Closed
Bug 1388514
Opened 7 years ago
Closed 7 years ago
Sidebar scrollbar is not clickable
Categories
(Firefox :: General, defect)
Tracking
()
VERIFIED
FIXED
Firefox 57
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox55 | --- | unaffected |
firefox56 | + | fixed |
firefox57 | --- | verified |
People
(Reporter: mat, Assigned: dao)
References
Details
(Keywords: regression)
Attachments
(2 files)
299.72 KB,
video/x-matroska
|
Details | |
59 bytes,
text/x-review-board-request
|
Gijs
:
review+
gchang
:
approval-mozilla-beta+
|
Details |
Build identifier: Mozilla/5.0 (X11; Linux x86_64; rv:57.0) Gecko/20100101 Firefox/57.0 STR: - Using latest nightly with Ubuntu, open the sidebar - Make sidebar content big enough to need a scrollbar (e.g. open the history sidebar and then browse to enough pages for the scrollbar to appear) - Try to click on the sidebar's scrollbar Expected results: - Clicking on the sidebar's scrollbar is possible and the position inside the sidebar content changes accordingly Actual results: - Clicking on the sidebar's scrollbar is not possible. Instead, you either get a resize handle or end up clicking on the contents of the sidebar itself (in the case of the history sidebar, links) Note: scrolling inside the sidebar with the mousewheel works fine. Clicking on the page scrollbar (or any other) instead also appears to work fine, only the sidebar's scrollbar is affected. This is happening reliably for me using Ubuntu 16.04 LTS with the default theme and GNOME Desktop (I don't have easy access to a Mac or Windows to test there). I can *not* reproduce with Firefox 55.0.
Reporter | ||
Comment 1•7 years ago
|
||
Tested with mozregression and the problem seems to have started with https://hg.mozilla.org/mozilla-central/rev/b77dc8c1e68f from bug 1377070
Comment 2•7 years ago
|
||
Same setup, except ubuntu 17.04, no issue on my sidebar.
Comment 3•7 years ago
|
||
(In reply to Mathieu Pillard [:mat] from comment #1) > Tested with mozregression and the problem seems to have started with > https://hg.mozilla.org/mozilla-central/rev/b77dc8c1e68f from bug 1377070 Dão, can you take a look?
Blocks: 1377070
Flags: needinfo?(dao+bmo)
Updated•7 years ago
|
status-firefox56:
--- → affected
tracking-firefox56:
--- → ?
Comment 4•7 years ago
|
||
FWIW, I can reproduce, and only the leftmost pixel of the scrollbar is still clickable for me. Maybe we just need to back this out, or ensure that there's a border on the sidebar that is sufficiently large that the click target for the resizer doesn't interfere with the scrollbar.
Updated•7 years ago
|
Keywords: regression
Comment 9•7 years ago
|
||
This should be blocker for Firefox56, Because It is unable to scroll on Gnome Unity desktop.
Severity: normal → blocker
Assignee | ||
Comment 10•7 years ago
|
||
(In reply to Alice0775 White from comment #9) > This should be blocker for Firefox56, Because It is unable to scroll on > Gnome Unity desktop. The severity field doesn't work like that.
Assignee: nobody → dao+bmo
Severity: blocker → normal
Status: NEW → ASSIGNED
Flags: needinfo?(dao+bmo)
Comment hidden (mozreview-request) |
Comment 12•7 years ago
|
||
mozreview-review |
Comment on attachment 8905536 [details] Bug 1388514 - Make the sidebar splitter narrower again to prevent it from overlapping the scrollbar too much, and disable this behavior on Linux since scrollbars are too narrow on Ubuntu. https://reviewboard.mozilla.org/r/177334/#review182402 r=me with a comment added. ::: browser/themes/shared/sidebar.inc.css:14 (Diff revision 1) > font-size: 1.333em; > font-weight: lighter; > padding: 8px; > } > > +%ifndef MOZ_WIDGET_GTK Can you add a % comment here explaining why we don't do this on Linux?
Attachment #8905536 -
Flags: review?(gijskruitbosch+bugs) → review+
Comment hidden (mozreview-request) |
Comment 14•7 years ago
|
||
Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1c37a53a0c8f Make the sidebar splitter narrower again to prevent it from overlapping the scrollbar too much, and disable this behavior on Linux since scrollbars are too narrow on Ubuntu. r=Gijs
Comment 15•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/1c37a53a0c8f
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Comment 16•7 years ago
|
||
Comment on attachment 8905536 [details] Bug 1388514 - Make the sidebar splitter narrower again to prevent it from overlapping the scrollbar too much, and disable this behavior on Linux since scrollbars are too narrow on Ubuntu. Approval Request Comment [Feature/Bug causing the regression]: bug 1377070 [User impact if declined]: scrolling the sidebar is hard/impossible, because trying to drag the scrollbar resizes the sidebar instead. [Is this code covered by automated tests?]: no, I don't think our existing test framework really allow for testing mouse interactivity with scrollbars in a way that would catch this. [Has the fix been verified in Nightly?]: not yet [Needs manual test from QE? If yes, steps to reproduce]: 1. open firefox 2. open bookmarks or history sidebar 3. ensure there's enough content there (create and copy/paste to duplicate some bookmarks, or browse to a few pages to create history) to create a vertical scrollbar (it may also help to reduce the height of the browser window) 4. try to use the scrollbar by dragging it with the mouse [List of other uplifts needed for the feature/fix]: n/a [Is the change risky?]: no [Why is the change risky/not risky?]: mostly reverting a previous patch, styling only changes. [String changes made/needed]: nope
Attachment #8905536 -
Flags: approval-mozilla-beta?
Comment 17•7 years ago
|
||
Hi Florin, could you help find someone to verify if this issue was fixed as expected on the latest Nightly build? Thanks!
Flags: qe-verify+
Flags: needinfo?(florin.mezei)
Comment 18•7 years ago
|
||
Tested on latest Nightly 57.0a1 Build ID 20170908100218 Mozilla/5.0 (X11; Linux x86_64; rv:57.0) Gecko/20100101 Firefox/57.0 and the issue is verified-fixed and the expected requirements are met as following: - Now, clicking on the vertical scrollbar it's possible without any issues detected and the position inside the sidebar will change the content accordingly if dragged, using the mouse. - The scrollbar is wide enough to not interfere with the content when focused with the mouse - The border on the sidebar is sufficiently large, that the click target for the resizer doesn't interfere with the scrollbar Also, i can confirm that the issue was reproducible on Nightly 57 ID 20170808100224 Ubuntu 16.04 x 64 Looks good. Thank you
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Flags: needinfo?(florin.mezei)
Updated•7 years ago
|
status-firefox55:
--- → unaffected
status-firefox-esr52:
--- → unaffected
Version: 57 Branch → 56 Branch
Comment 19•7 years ago
|
||
Comment on attachment 8905536 [details] Bug 1388514 - Make the sidebar splitter narrower again to prevent it from overlapping the scrollbar too much, and disable this behavior on Linux since scrollbars are too narrow on Ubuntu. Fix a regression and was verified. Beta56+.
Attachment #8905536 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 20•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/5493550a0575
You need to log in
before you can comment on or make changes to this bug.
Description
•