Closed Bug 1447752 Opened 7 years ago Closed 7 years ago

Search bar colour fill is wrong in Nightly

Categories

(Firefox :: New Tab Page, defect, P1)

61 Branch
defect

Tracking

()

VERIFIED FIXED
Firefox 61
Iteration:
61.2 - Apr 9
Tracking Status
firefox60 + verified
firefox61 --- verified

People

(Reporter: amylee, Assigned: rrosario)

References

Details

User Story

https://github.com/mozilla/activity-stream/compare/9a8627f9feb0747f85b2dbc3cf6bdf616829ebfc...firefox-60b10

Attachments

(2 files)

Attached image Screen Shot 2018-03-20 at 17.16.09.png (deleted) —
The fill colour for the search bar should be #FFFFFF. Right now it's the same colour as the New Tab background colour (#f9f9fa) so it looks disabled.
Ah doh, pretty sure this regressed from bug 1418090. https://github.com/mozilla/activity-stream/pull/3982
Depends on: 1418090
Assignee: nobody → rrosario
Iteration: --- → 61.2 - Apr 9
Priority: -- → P1
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Blocks: 1449792
[Tracking Requested - why for this release]: phlsa says "I’m a bit worried that this will look like it’s disabled to users, causing them to not search there." … "Then we just need to make sure that the fix also gets uplifted to beta"
Please request Beta approval on this when you get a chance.
Flags: needinfo?(rrosario)
r1cky, we'll need a new patch for beta 60 as we probably won't uplift dark theme. Could you put together a PR against https://github.com/mozilla/activity-stream/tree/firefox-60
User Story: (updated)
Depends on: 1448394
Flags: qe-verify+
Build ID: 20180402111808 User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:61.0) Gecko/20100101 Firefox/61.0 I have verified that this issue is no longer reproducible on Windows 10 x64, Arch Linux and Mac 10.13.3 with the latest Nightly build (61.0a1) installed. Now the search bar color is displayed correctly.
Status: RESOLVED → VERIFIED
Attachment #8964379 - Flags: review?(edilee) → review+
Comment on attachment 8964379 [details] Bug 1447752 - Search bar colour fill is wrong in Nightly. Approval Request Comment [Feature/Bug causing the regression]: Undesired color set in bug 1418090 [User impact if declined]: Gray background instead of white could lead users to think searching is disabled [Is this code covered by automated tests?]: No [Has the fix been verified in Nightly?]: Yes, comment 7 [Needs manual test from QE? If yes, steps to reproduce]: No [List of other uplifts needed for the feature/fix]: This needs to land after bug 1448394 just as an artifact of how we export changes. [Is the change risky?]: No [Why is the change risky/not risky?]: One line css change setting an explicit color. The exported change is large due to per-platform styles and source .map files, but the original source change is small: https://github.com/mozilla/activity-stream/commit/2c94813f73db427d3ffcbd47534cbd23731b7407 [String changes made/needed]: None
Flags: needinfo?(rrosario)
Attachment #8964379 - Flags: approval-mozilla-beta?
Comment on attachment 8964379 [details] Bug 1447752 - Search bar colour fill is wrong in Nightly. activity stream regression fix, verified in nightly, approved for 60.0b10
Attachment #8964379 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
I managed to reproduce the initial issue on 61.0a1 (2018-03-21). I can confirm that the 60.0b10 (20180404171943) build is verified fixed across platforms (Windows 10 x64, Ubuntu 16.04 x64 and macOS 10.13.3). The search bar is now properly displayed and its color is #FFF, in both normal and highlighted modes.
Flags: qe-verify+
User Story: (updated)
Depends on: 1454743
Blocks: 1454743
No longer depends on: 1454743
Component: Activity Streams: Newtab → New Tab Page
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: