The onboarding notification button should not wrap

VERIFIED FIXED in Firefox 56

Status

()

Firefox
General
P1
normal
VERIFIED FIXED
a month ago
16 days ago

People

(Reporter: verdi, Assigned: rickychien)

Tracking

53 Branch
Firefox 56
Points:
---
Bug Flags:
qe-verify +

Firefox Tracking Flags

(firefox56 verified)

Details

(Whiteboard: [photon-onboarding])

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

(Reporter)

Description

a month ago
Created attachment 8886899 [details]
learnmore.gif

The "Learn More" button in the onboarding notifications wrap when you shrink the window width. This isn't necessary and looks bad. We should prevent it from wrapping.

Updated

a month ago
Whiteboard: [photon-onboarding] → [photon-onboarding][triage]

Updated

a month ago
Assignee: nobody → fliu

Updated

a month ago
Flags: qe-verify+
Priority: -- → P1
QA Contact: jwilliams
Target Milestone: --- → Firefox 56

Updated

a month ago
Flags: needinfo?(rchien)
(Assignee)

Updated

a month ago
Assignee: fliu → rchien
Status: NEW → ASSIGNED
Flags: needinfo?(rchien)

Updated

a month ago
Whiteboard: [photon-onboarding][triage] → [photon-onboarding]
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
(Assignee)

Comment 3

a month ago
In order to avoid introducing a long button for certain specific language with using `white-space: nowrap`, we'd like to give a longer enough width `min-width: 130px` for fixing the text wrap issue at least for English.

Comment 4

29 days ago
mozreview-review
Comment on attachment 8888186 [details]
Bug 1381356 - The onboarding notification button should not wrap

https://reviewboard.mozilla.org/r/159118/#review164804
Attachment #8888186 - Flags: review?(dtownsend) → review+

Comment 5

29 days ago
Pushed by rchien@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/15085feea2b2
The onboarding notification button should not wrap r=mossop

Comment 6

28 days ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/15085feea2b2
Status: ASSIGNED → RESOLVED
Last Resolved: 28 days ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
I have reproduced this bug with nightly 56.0a1 (2017-07-16) on Linux Mint (64 Bit).

The bug's fix is now verified on Latest Nightly 56.0a1

Build ID 	20170727100240
User Agent 	Mozilla/5.0 (X11; Linux x86_64; rv:56.0) Gecko/20100101 Firefox/56.0
QA Whiteboard: [bugday-20170726]

Comment 8

22 days ago
I have successfully reproduced this bug with Nightly 56.0a1 (2017-07-16)  on windows 10(32bit)

this bug is verified fix with  latest nightly 56.0a1 (2017-07-27) (32-bit)

Build ID: 20170727100347
Mozilla/5.0 (Windows NT 10.0; rv:56.0) Gecko/20100101 Firefox/56.0

[bugday-20170726]

Updated

16 days ago
Status: RESOLVED → VERIFIED
status-firefox56: fixed → verified
You need to log in before you can comment on or make changes to this bug.