Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Scrolling makes back button grow on MacBook Pro #14785

Closed
vaaralav opened this issue Jul 19, 2018 · 13 comments
Closed

Scrolling makes back button grow on MacBook Pro #14785

vaaralav opened this issue Jul 19, 2018 · 13 comments

Comments

@vaaralav
Copy link

vaaralav commented Jul 19, 2018

Description

Scrolling on Mac OS by swiping up/down with two fingers on trackpad makes back button grow.

Steps to Reproduce

  1. Open any web page that is long enough to scroll
  2. Scroll up or down with the trackpad

Actual result:

Back button in the toolbar grows like the swipe gesture was back, not scroll. If the back button is in disabled state it will dim instead of growing.

jul-19-2018 09-21-26

Expected result:
The back button shouldn't react to scroll.

Reproduces how often:

Every tab.

Brave Version

about:brave info:

Brave: 0.23.39
V8: 6.7.288.46
rev: 3ee14b0
Muon: 7.1.6
OS Release: 17.6.0
Update Channel: Release
OS Architecture: x64
OS Platform: macOS
Node.js: 7.9.0
Brave Sync: v1.4.2
libchromiumcontent: 67.0.3396.103

@odelr005
Copy link

I have the same issue on all sites, on some sites however, it refuses to scroll at all.

@Alino
Copy link

Alino commented Jul 22, 2018

it's not a bug it's a feature 😆
For me there is a bug on my mac pro early 2015, that I don't have this feature.
MacOS 10.13.6

Name Version
Brave 0.23.39
V8 6.7.288.46
rev 3ee14b0
Muon 7.1.6
OS Release 17.7.0
Update Channel Release
OS Architecture x64
OS Platform macOS
Node.js 7.9.0
Brave Sync v1.4.2
libchromiumcontent 67.0.3396.103

@bsclifton bsclifton added the bug label Jul 22, 2018
@bsclifton bsclifton added this to the Triage Backlog milestone Jul 22, 2018
@bsclifton
Copy link
Member

@vaaralav @odelr005 can you try editing the sensitivity under Swipe Navigation Distance? Maybe turn this up a little bit (move to the right)
screen shot 2018-07-22 at 9 15 24 am

@bsclifton bsclifton added the needs-info Another team member needs information from the PR/issue opener. label Jul 22, 2018
@GalacticSun
Copy link

I'm also having this problem. The "Swipe Navigation Distance" helps a little bit, but not 100%. The fact that vertical scrolling affects the size of the Forward / Back buttons is bad. Users shouldn't have to manage their vertical scrolling dexterity to ensure the browser doesn't suddenly jump Back / Forward ;) I love Brave, though, keep up the great work, and thanks!

@bsclifton
Copy link
Member

cc: @petemill

@odelr005
Copy link

Also when viewing pages that require vertical scrolling, if the gesture is done to scroll vertically, the browser will easily go forwards or backwards a page. The "Swipe Navigation Distance" does not seem to help much unless you swipe extremely fast. The faster you swipe the less the issue is noticed.

@joelwass
Copy link

It seems to only affect my browser once I begin swiping left or right. If i exclusively swipe vertically, it doesn't affect the size of the back button, but once I've per se "broken the seal" of swiping even slightly horizontally, the button reacts to vertical swiping.

joelwass added a commit to joelwass/browser-laptop that referenced this issue Jul 24, 2018
@bsclifton bsclifton modified the milestones: Triage Backlog, Completed work Jul 26, 2018
@TeoMorabito
Copy link

When I scroll horizontally on a website such as a calendar, scrolling sideways makes the page go back of forward in history.

If content is scrollable horizontally, nav scroll should be disabled, or at least a setting to disable it.

Using a retina macbook pro

@joelwass
Copy link

Hey @TeoMorabito , that sounds like a separate issue to me. Do you want to open another issue for it and tag me in it and I can take a look? I'm assuming that would mean just disabling nav scroll when the content on the page is horizontally scrollable.

@amclin
Copy link

amclin commented Aug 5, 2018

I can replicate this zooming of the back button consistently. Pushing the "Swipe Navigation Distance" bar to the max has no effect other than to make the zoom animation a little jerky and not always go to maximum size.

Brave: 0.23.77
V8: 6.8.275.24
rev: 0125b5f
Muon: 8.0.6
OS Release: 16.7.0
Update Channel: Release
OS Architecture: x64
OS Platform: macOS
Node.js: 7.9.0
Brave Sync: v1.4.2
libchromiumcontent: 68.0.3440.84

@TeoMorabito
Copy link

Going into System Preference > Trackpad
and unchecking swipe between pages disables the swipe right and left gesture and thus disables the navigation swipe in brave.
(I am not able to swipe horizontally on a calendar so needed a way to disable it so it wouldn't go back and forth in the history)

darkdh added a commit that referenced this issue Aug 30, 2018
only fire once each scroll since C69

mainly
fix #15114

also
fix #14871
fix #14829
fix #14785

Auditor: @bsclifton
@darkdh darkdh modified the milestones: Completed work, 0.23.x w/ Chromium 69 (BETA Channel) Aug 30, 2018
@darkdh
Copy link
Member

darkdh commented Aug 30, 2018

fixed by 69851d1

@darkdh darkdh closed this as completed Aug 30, 2018
darkdh added a commit that referenced this issue Aug 30, 2018
only fire once each scroll since C69

mainly
fix #15114

also
fix #14871
fix #14829
fix #14785

Auditor: @bsclifton
@bsclifton bsclifton removed the needs-info Another team member needs information from the PR/issue opener. label Aug 30, 2018
bsclifton pushed a commit that referenced this issue Aug 31, 2018
only fire once each scroll since C69

mainly
fix #15114

also
fix #14871
fix #14829
fix #14785

Auditor: @bsclifton
@LaurenWags
Copy link
Member

LaurenWags commented Sep 4, 2018

Verified with macOS 10.12.6 using

  • 0.23.203 16c2c19
  • Muon 8.1.2
  • libchromiumcontent 69.0.3497.72

Verified with macOS 10.13.6 using the following build:

  • 0.23.203 16c2c19
  • Muon: 8.1.2
  • libchromiumcontent: 69.0.3497.72

Before going through verification, ensured I could reproduce the issue using 0.23.105.

bsclifton pushed a commit that referenced this issue Sep 11, 2018
only fire once each scroll since C69

mainly
fix #15114

also
fix #14871
fix #14829
fix #14785

Auditor: @bsclifton
bsclifton pushed a commit that referenced this issue Sep 14, 2018
only fire once each scroll since C69

mainly
fix #15114

also
fix #14871
fix #14829
fix #14785

Auditor: @bsclifton
bsclifton pushed a commit that referenced this issue Sep 18, 2018
only fire once each scroll since C69

mainly
fix #15114

also
fix #14871
fix #14829
fix #14785

Auditor: @bsclifton
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.