Closed Bug 1162274 Opened 10 years ago Closed 10 years ago

[Search] "Settings" in search suggestion dialog is not linked to the settings screen

Categories

(Firefox OS Graveyard :: Gaia::Search, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-b2g:2.5+, b2g-v2.2 unaffected, b2g-master verified)

VERIFIED FIXED
2.2 S12 (15may)
blocking-b2g 2.5+
Tracking Status
b2g-v2.2 --- unaffected
b2g-master --- verified

People

(Reporter: ychung, Assigned: stas)

References

()

Details

(Keywords: regression, Whiteboard: [3.0-Daily-Testing][systemsfe])

Attachments

(4 files)

Attached file logcat_20150506_1449.txt (deleted) —
Description: On Flame Master, the "Settings" on the search suggestion dialog is not highlighted and linked to the settings page. Pre-requisite: Have an internet connection. Repro Steps: 1) Update a Flame to 20150506010204. 2) Tap the rocketbar, and type in a few words. 3) Observe the dialog "Search suggestions are enabled. The characters you type when....." Actual: "Settings" at the end of the dialog is not highlighted and linked to the settings page. Expected: "Settings" at the end of the dialog is highlighted and linked to the settings page. Environmental Variables: Device: Flame 3.0 (KK, 319mb, full flash) Build ID: 20150506010204 Gaia: 3e6fd1e0a478af2c95d09ce95c2c6de2de2fec14 Gecko: ba44099cbd07 Gonk: a9f3f8fb8b0844724de32426b7bcc4e6dc4fa2ed Version: 40.0a1 (3.0) Firmware Version: v18D-1 User Agent: Mozilla/5.0 (Mobile; rv:40.0) Gecko/40.0 Firefox/40.0 Repro frequency: 7/7 See attached: screenshot, video clip, logcat https://youtu.be/VAcALfQFpZk?t=24s
This issue does NOT reproduce on Flame 2.2. Result: "Settings" at the end of the dialog is highlighted and linked to the settings page. Environmental Variables: Device: Flame 2.2 (KK, 319mb, full flash) Build ID: 20150506002501 Gaia: 772a9491909abd02dc67278dd453746e2dd358a8 Gecko: 3af6a0a79227 Gonk: ab265fb203390c70b8f2a054f38cf4b2f2dad70a Version: 37.0 (2.2) Firmware Version: v18D-1 User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Attached image LinkToSettings.png (deleted) —
Can we get a regression window here?
[Blocking Requested - why for this release]: To my knowledge this has always worked so nominating this 3.0? since we regressed here.
blocking-b2g: --- → 3.0?
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
QA Contact: ychung
b2g-inbound Regression Window: Last Working Environmental Variables: Device: Flame 3.0 BuildID: 20150427103815 Gaia: e8f4231141210db01f43405e0c0336f480083ae7 Gecko: fce6aac57f06 Version: 40.0a1 (3.0) Firmware Version: v18D-1 User Agent: Mozilla/5.0 (Mobile; rv:40.0) Gecko/40.0 Firefox/40.0 First Broken Environmental Variables: Device: Flame 3.0 BuildID: 20150427110514 Gaia: 5fd082cb265885248974fbb4ff70239901a6427e Gecko: 0a66e85d0a42 Version: 40.0a1 (3.0) Firmware Version: v18D-1 User Agent: Mozilla/5.0 (Mobile; rv:40.0) Gecko/40.0 Firefox/40.0 Last Working Gaia First Broken Gecko: Issue does NOT reproduce Gaia: e8f4231141210db01f43405e0c0336f480083ae7 Gecko: 0a66e85d0a42 First Broken Gaia Last Working Gecko: Issue DOES reproduce Gaia: 5fd082cb265885248974fbb4ff70239901a6427e Gecko: fce6aac57f06 https://github.com/mozilla-b2g/gaia/compare/e8f4231141210db01f43405e0c0336f480083ae7...5fd082cb265885248974fbb4ff70239901a6427e Caused by bug 1142526
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
QA Contact: ychung
Staś, can you take a look at this please? This might have been caused by the landing for bug 1142526.
Blocks: 1142526
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker) → needinfo?(stas)
Not sure if it's related, but I noticed this in build log today: [/build_stage/search/index.html: L10n Deprecation Warning: using innerHTML in translations is unsafe and will not be supported in future versions of l10n.js. See https://bugzil.la/1027117 ] (as a matter of fact there are two warnings like this, another for FTU).
Comment on attachment 8602666 [details] [gaia] stasm:1162274-settings-link > mozilla-b2g:master Thanks for flagging this. It was caused by the fact that l10n.js doesn't allow the "id" attribute in translations any more. The fix is to move the id to the source HTML. Together with bug 1159906, this was the only instance of "id" or "class" used in translations in Gaia.
Flags: needinfo?(stas)
Attachment #8602666 - Flags: review?(dale)
Comment on attachment 8602666 [details] [gaia] stasm:1162274-settings-link > mozilla-b2g:master This looks good, thanks
Attachment #8602666 - Flags: review?(dale) → review+
Keywords: checkin-needed
https://github.com/mozilla-b2g/gaia/pull/29943 Autolander could not land the pull request due to not having collaborator rights. This is possibly due to a tree closure. Please check the tree status and request checkin again once the tree is open.
blocking-b2g: 3.0? → 3.0+
Keywords: checkin-needed
Keywords: checkin-needed
https://github.com/mozilla-b2g/gaia/pull/29943 Autolander could not land the pull request due to not having collaborator rights. This is possibly due to a tree closure. Please check the tree status and request checkin again once the tree is open.
Assignee: nobody → stas
The tree was closed, but adding checkin-needed to trigger autolander again.
Keywords: checkin-needed
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Blocks: 1142313
NI myself to verify the bug tomorrow and re-visit bug 1142313.
Flags: needinfo?(ychung)
Keywords: verifyme
This bug has been verified as pass on latest Nightly build of Flame v3.0 and Nexus 5 v3.0 by the STR in Comment 0. Actual results: The "Settings" word on the search suggestion dialog is highlighted and linked to the Settings page. See attachment: verified_v3.0.mp4 Reproduce rate: 0/5 Device: Flame 3.0 build(Pass) Build ID 20150510010201 Gaia Revision 5b2a150f6f5d29bddfaac13fcbbf099376f2f275 Gaia Date 2015-05-09 12:34:41 Gecko Revision https://hg.mozilla.org/mozilla-central/rev/77d92f6d7679 Gecko Version 40.0a1 Device Name flame Firmware(Release) 4.4.2 Firmware(Incremental) eng.cltbld.20150510.045933 Firmware Date Sun May 10 04:59:44 EDT 2015 Bootloader L1TC000118D0 Device: Nexus 5 v3.0 build (Pass) Build ID 20150510010201 Gaia Revision 5b2a150f6f5d29bddfaac13fcbbf099376f2f275 Gaia Date 2015-05-09 12:34:41 Gecko Revision https://hg.mozilla.org/mozilla-central/rev/77d92f6d7679 Gecko Version 40.0a1 Device Name hammerhead Firmware(Release) 5.1 Firmware(Incremental) eng.cltbld.20150510.044405 Firmware Date Sun May 10 04:44:24 EDT 2015 Bootloader HHZ12f
Attached video verified_v3.0.mp4 (deleted) —
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage+][MGSEI-Triage+]
Status: RESOLVED → VERIFIED
Flags: needinfo?(ychung)
Blocks: 1163825
No longer blocks: 1163825
Target Milestone: --- → 2.2 S12 (15may)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: