Closed
Bug 1225054
Opened 8 years ago
Closed 8 years ago
In room list, if a room name is too long and requires "...", the "..." renders over the menu items of the pen menu
Categories
(Hello (Loop) :: Client, defect, P2)
Hello (Loop)
Client
Tracking
(firefox45 fixed)
RESOLVED
FIXED
mozilla45
Tracking | Status | |
---|---|---|
firefox45 | --- | fixed |
People
(Reporter: RT, Assigned: mancas)
References
Details
Attachments
(3 files)
119.70 KB,
image/png
|
Details | |
870 bytes,
patch
|
mikedeboer
:
review+
|
Details | Diff | Splinter Review |
108.65 KB,
image/png
|
Pau
:
ui-review+
|
Details |
No description provided.
Reporter | ||
Comment 1•8 years ago
|
||
Snapshot of the issue
Reporter | ||
Updated•8 years ago
|
Rank: 21
Priority: -- → P2
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → b.mcb
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•8 years ago
|
||
Mike, can you review the patch? it should be quick. I've just added a css rule =) Thanks!
Attachment #8687932 -
Flags: review?(mdeboer)
Assignee | ||
Comment 3•8 years ago
|
||
Attachment #8687933 -
Flags: ui-review?(b.pmm)
Updated•8 years ago
|
Attachment #8687933 -
Flags: ui-review?(b.pmm) → ui-review+
Comment 4•8 years ago
|
||
(In reply to Manuel Casas Barrado [:mancas] from comment #2) > Mike, can you review the patch? it should be quick. I've just added a css > rule =) > > Thanks! Sure, but what's the cause of the ellipsis showing on top of the dropdown?
Flags: needinfo?(b.mcb)
Assignee | ||
Comment 5•8 years ago
|
||
If we don't set a z-index, the default value for the rule will be "auto", and according to the w3c: The stack level of the generated box in the current stacking context is 0. The box does not establish a new stacking context unless it is the root element. [1] So the ellipsis is in the same stack level. [1] http://www.w3.org/TR/CSS2/visuren.html#z-index
Flags: needinfo?(b.mcb) → needinfo?(mdeboer)
Comment 6•8 years ago
|
||
I understand how z-index works :-) No, what I meant was that I don't understand that the absolutely positioned element (dropdown) would render on top of the text, except for the ellipsis which is there because of `text-overflow: ellipsis`. This must be a platform issue.
Flags: needinfo?(mdeboer)
Comment 7•8 years ago
|
||
Comment on attachment 8687932 [details] [diff] [review] In room list, if a room name is too long and requires "...", the "..." renders over the menu items of the pen menu. Review of attachment 8687932 [details] [diff] [review]: ----------------------------------------------------------------- Seems like this work around is necessary! Thanks, Manu.
Attachment #8687932 -
Flags: review?(mdeboer) → review+
Assignee | ||
Comment 8•8 years ago
|
||
Ah! Maybe you're right and this is a platform issue, because the ellipsis is generated and rendered by the browser =)
Keywords: checkin-needed
Comment 10•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/055104774f5d
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
You need to log in
before you can comment on or make changes to this bug.
Description
•