Closed
Bug 1386560
Opened 7 years ago
Closed 7 years ago
The Firefox Account header should not be highlighted on search result page.
Categories
(Firefox :: Settings UI, defect, P1)
Tracking
()
VERIFIED
FIXED
Firefox 57
People
(Reporter: evanxd, Assigned: evanxd)
References
Details
(Whiteboard: [photon-preference])
Attachments
(1 file)
(deleted),
text/x-review-board-request
|
jaws
:
review+
lizzard
:
approval-mozilla-beta+
|
Details |
The class attribute for the firefoxAccountCategory header is not correct. It supports to be "subcategory" not "searchCategory". It is a typo.
Flags: qe-verify+
Comment hidden (mozreview-request) |
Assignee | ||
Comment 2•7 years ago
|
||
The STR:
1. Input the "Account" keyword in search input to search preferences.
Actual:
The "Firefox Account" on the Firefox Account pane is highlighted.
Expected:
Should not show the "Firefox Account" header since we make the decision in Bug 1382134.
Summary: The class attribute for the firefoxAccountCategory header is not correct → The Firefox Account header should not be highlighted on search result page.
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Attachment #8892816 -
Flags: review?(jaws)
Assignee | ||
Comment 4•7 years ago
|
||
Hi Jared,
Could you help review the patch?
You could see the STR at Comment 2.
And I think we should uplift this to beta 56.
Thank you.
Assignee | ||
Updated•7 years ago
|
Status: NEW → ASSIGNED
Updated•7 years ago
|
status-firefox56:
--- → affected
status-firefox57:
--- → affected
Updated•7 years ago
|
Whiteboard: [photon-preference][triage] → [photon-preference]
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8892816 [details]
Bug 1386560 - Fix the typo of firefoxAccountCategory header's class list because all level 1 headers should have the subcategory class.
https://reviewboard.mozilla.org/r/163800/#review170032
Attachment #8892816 -
Flags: review?(jaws) → review+
Assignee | ||
Comment 6•7 years ago
|
||
Thank you for reviewing, Jared. Let's land it.
Keywords: checkin-needed
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 7•7 years ago
|
||
The patch will be landed after the try[1] is good.
[1]: https://treeherder.mozilla.org/#/jobs?repo=try&revision=9185d302eb4a
Assignee | ||
Comment 8•7 years ago
|
||
Comment on attachment 8892816 [details]
Bug 1386560 - Fix the typo of firefoxAccountCategory header's class list because all level 1 headers should have the subcategory class.
Approval Request Comment
[Feature/Bug causing the regression]: Bug 1365133
[User impact if declined]: The "Firefox Account" header will be shown in search result page if search "Account" but we don't want it happen.
[Is this code covered by automated tests?]: No.
[Has the fix been verified in Nightly?]: Not yet.
[Needs manual test from QE? If yes, steps to reproduce]: Yes, please refer to https://bugzilla.mozilla.org/show_bug.cgi?id=1386560#c2 to learn the STR.
[List of other uplifts needed for the feature/fix]: None.
[Is the change risky?]: No.
[Why is the change risky/not risky?]: This is one line xul change, no code change.
[String changes made/needed]: No.
Attachment #8892816 -
Flags: approval-mozilla-beta?
Assignee | ||
Comment 9•7 years ago
|
||
The try[1] looks good. Let's land it.
[1]: https://treeherder.mozilla.org/#/jobs?repo=try&revision=9185d302eb4a
Keywords: checkin-needed
Comment 10•7 years ago
|
||
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/e65dc2bcc867
Fix the typo of firefoxAccountCategory header's class list because all level 1 headers should have the subcategory class. r=jaws
Keywords: checkin-needed
Comment 11•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Comment on attachment 8892816 [details]
Bug 1386560 - Fix the typo of firefoxAccountCategory header's class list because all level 1 headers should have the subcategory class.
Simple fix, let's bring it to 56 (for beta 1)
Attachment #8892816 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•7 years ago
|
Target Milestone: Firefox 56 → Firefox 57
Comment 13•7 years ago
|
||
bugherder uplift |
Comment 14•7 years ago
|
||
Build ID; 20170806100257
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:57.0) Gecko/20100101 Firefox/57.0
Verified as fixed on Firefox Nightly 57.0a1 on Windows 10 x 64, Mac OS X 10.12 and Ubuntu 16.04 x64.
Comment 15•7 years ago
|
||
Build ID: 20170808030909
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:56.0) Gecko/20100101 Firefox/56.0
Verified as fixed on Firefox Beta 56.0b1 on Windows 10 x 64, Mac OS X 10.12 and Ubuntu 16.04 x64.
Status: RESOLVED → VERIFIED
Comment 16•7 years ago
|
||
https://hg.mozilla.org/projects/date/rev/e65dc2bcc867930d4cadea96aa7c2907b5eea163
Bug 1386560 - Fix the typo of firefoxAccountCategory header's class list because all level 1 headers should have the subcategory class. r=jaws
You need to log in
before you can comment on or make changes to this bug.
Description
•