If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Sidebar scrollbar is not clickable

VERIFIED FIXED in Firefox 56

Status

()

Firefox
General
VERIFIED FIXED
2 months ago
11 days ago

People

(Reporter: mat, Assigned: dao)

Tracking

({regression})

56 Branch
Firefox 57
Unspecified
Linux
regression
Points:
---

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox55 unaffected, firefox56+ fixed, firefox57 verified)

Details

MozReview Requests

()

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

Attachments

(2 attachments)

(Reporter)

Description

2 months ago
Created attachment 8895065 [details]
Video of the issue

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

2 months ago
Tested with mozregression and the problem seems to have started with https://hg.mozilla.org/mozilla-central/rev/b77dc8c1e68f from bug 1377070
Same setup, except ubuntu 17.04, no issue on my sidebar.

Comment 3

a month 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

a month ago
status-firefox56: --- → affected
tracking-firefox56: --- → ?

Comment 4

a month 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.
Track 56+ as this UI behavior needs to be polished.
tracking-firefox56: ? → +

Updated

a month ago
Duplicate of this bug: 1390690

Updated

15 days ago
Duplicate of this bug: 1397694

Updated

15 days ago
Keywords: regression

Updated

15 days ago
Duplicate of this bug: 1397516

Comment 9

15 days ago
This should be blocker for Firefox56, Because It is unable to scroll on Gnome Unity desktop.
Severity: normal → blocker
(Assignee)

Comment 10

15 days 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

15 days 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

15 days 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
https://hg.mozilla.org/mozilla-central/rev/1c37a53a0c8f
Status: ASSIGNED → RESOLVED
Last Resolved: 15 days ago
status-firefox57: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57

Comment 16

15 days 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?
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

14 days 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
status-firefox57: fixed → verified
Flags: qe-verify+
Flags: needinfo?(florin.mezei)
status-firefox55: --- → unaffected
status-firefox-esr52: --- → unaffected
Version: 57 Branch → 56 Branch
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

11 days ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-beta/rev/5493550a0575
status-firefox56: affected → fixed
You need to log in before you can comment on or make changes to this bug.