Closed
Bug 1093056
Opened 9 years ago
Closed 9 years ago
Scrollbars shown in room list when there are no rooms
Categories
(Hello (Loop) :: Client, defect, P1)
Hello (Loop)
Client
Tracking
(firefox35 fixed, firefox36 fixed)
People
(Reporter: standard8, Assigned: standard8)
References
Details
(Whiteboard: [good first verify])
Attachments
(2 files)
13.78 KB,
image/png
|
Details | |
762 bytes,
patch
|
NiKo
:
review+
lsblakk
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
When I have no rooms in the list, I get a strange box with a couple of empty scrollbars in it. This looks strange - see the attached image.
Assignee | ||
Comment 1•9 years ago
|
||
Simple fix to stop scrollbars being shown permanently.
Attachment #8516597 -
Flags: review?(nperriault)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → standard8
Iteration: --- → 36.2
Priority: -- → P1
Comment on attachment 8516597 [details] [diff] [review] Scrollbars shouldn't be shown in the room list by default. Review of attachment 8516597 [details] [diff] [review]: ----------------------------------------------------------------- Make sense!
Attachment #8516597 -
Flags: review?(nperriault) → review+
Assignee | ||
Comment 3•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/d64ba06f8472
Target Milestone: --- → mozilla36
https://hg.mozilla.org/mozilla-central/rev/d64ba06f8472
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
status-firefox36:
--- → fixed
Flags: qe-verify-
Flags: in-testsuite?
Flags: in-moztrap-
Whiteboard: [good first verify]
status-firefox35:
--- → fixed
Comment 6•9 years ago
|
||
Comment on attachment 8516597 [details] [diff] [review] Scrollbars shouldn't be shown in the room list by default. Approval Request Comment Landed on aurora per IRC with lsblakk with a=loop-only
Attachment #8516597 -
Flags: approval-mozilla-aurora?
Updated•9 years ago
|
Attachment #8516597 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 7•8 years ago
|
||
css-only change, so I'm not sure this is worth looking at testsuite for. This was just a simple mistake that's easily noticeable visibly.
Flags: in-testsuite? → in-testsuite-
You need to log in
before you can comment on or make changes to this bug.
Description
•