Closed
Bug 1181684
Opened 9 years ago
Closed 9 years ago
Using a long email to sign into Sync screws up the layout of the hamburger menu.
Categories
(Firefox :: Sync, defect, P2)
Tracking
()
People
(Reporter: ckarlof, Assigned: eoger)
References
Details
(Whiteboard: [fxsync])
Attachments
(3 files)
288.98 KB,
image/png
|
Details | |
945 bytes,
patch
|
Gijs
:
review+
ritu
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
81.32 KB,
image/png
|
Details |
Screenshot attached.
Reporter | ||
Updated•9 years ago
|
Priority: -- → P2
Reporter | ||
Comment 1•9 years ago
|
||
eoger, it would be great to look at this while you're working on the hamburger menu stuff.
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → edouard.oger
Assignee | ||
Comment 2•9 years ago
|
||
Here's the patch for this.
Attachment #8631160 -
Flags: review?(markh)
Assignee | ||
Comment 3•9 years ago
|
||
And how it looks!
Comment 4•9 years ago
|
||
Comment on attachment 8631160 [details] [diff] [review] bug-1181684.patch Review of attachment 8631160 [details] [diff] [review]: ----------------------------------------------------------------- Gijs, This is a trivial patch, but my CSS foo is letting me down whether this is indeed the best way to solve this problem, so it would be great if you can have a look.
Attachment #8631160 -
Flags: review?(markh) → review?(gijskruitbosch+bugs)
Comment 5•9 years ago
|
||
Comment on attachment 8631160 [details] [diff] [review] bug-1181684.patch Review of attachment 8631160 [details] [diff] [review]: ----------------------------------------------------------------- Well, just messing with the flexbox styles here doesn't seem to do anything, which I'm going to chalk up to "hello, XUL, we hate you too." wfm!
Attachment #8631160 -
Flags: review?(gijskruitbosch+bugs) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Updated•9 years ago
|
Rank: 15
Updated•9 years ago
|
Iteration: --- → 42.1 - Jul 13
https://hg.mozilla.org/mozilla-central/rev/4b9160a4a8bc
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox42:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
Comment 8•9 years ago
|
||
Comment on attachment 8631160 [details] [diff] [review] bug-1181684.patch Approval Request Comment [Feature/regressing bug #]: 1139698 [User impact if declined]: Users with a very long email address will see the layout of the hamburger menu messed up. [Describe test coverage new/current, TreeHerder]: Existing tests pass [Risks and why]: None [String/UUID change made/needed]: None
Attachment #8631160 -
Flags: approval-mozilla-aurora?
Comment on attachment 8631160 [details] [diff] [review] bug-1181684.patch Looks safe and fixes a pretty visible UI bug. Approved.
Attachment #8631160 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
status-firefox41:
--- → affected
Updated•9 years ago
|
QA Contact: catalin.varga
Updated•9 years ago
|
Whiteboard: [fxsync]
Comment 11•9 years ago
|
||
Verified as fixed using the following environment: FF 41 Build Id: 20150804004026 FF 42 Build Id: 20150804030204 Os: Win 7 x64, Ubuntu 12.04 x86, Mac Os 10.10
You need to log in
before you can comment on or make changes to this bug.
Description
•