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

Fixed back/forward gesture to match system preferences #3299

Closed
apfelchips opened this issue Aug 21, 2016 · 9 comments · Fixed by #7786
Closed

Fixed back/forward gesture to match system preferences #3299

apfelchips opened this issue Aug 21, 2016 · 9 comments · Fixed by #7786

Comments

@apfelchips
Copy link

apfelchips commented Aug 21, 2016

Test plan

For swipe conflict

  1. Navigate to some sites and make sure you have back/forward history
  2. Open dev panel by shift + f8 to make webview smaller
  3. Go to https://github.com/brave/browser-laptop/projects/1
  4. Swipe left/back should not trigger navigation until reaching the edge

For swipe setting

  1. Navigate to some sites and make sure you have back/forward history
  2. Turn off "Swipe between edges" in system setting
  3. There should be any navigations by edge swiping
  4. Turn on "Swipe between edges" in systme setting
  5. There will be navigation by edge swiping

Original issue description

Experienced behavior:
I've tried to use the forward and backwards swipe gesture in Brave on my Macbook Pro trackpad but it never worked for me. I was quite puzzled because I've looked up if Brave supported macOS trackpad Gestures and read that in theory it should recognize them.

I later discovered that I've diverted from the default two finger "Swipe between pages" option in the Trackpad settings to three fingers.
screen shot 2016-08-22 at 00 10 25

When defaulting to the two finger option Brave oddly recognizes three finger back/forward gestures, but not the selected two finger ones, which doesn't really feel right.

Expected behavior:
Brave should adhere to the setting made in macOS and use two and three finger swipes according to the selected option, or at least support 3 finger swipes when they are selected in settings.

@apfelchips apfelchips changed the title Back/Forward Gesture doesn't work when "Swipe with 3 fingers" is enabled Back/Forward Gestures don't work when "Swipe with three fingers" is selected Aug 21, 2016
@bradleyrichter bradleyrichter added this to the 1.0.1 milestone Nov 21, 2016
@bradleyrichter
Copy link
Contributor

adding to this: We need to base the fixes against Safari for swipe nav vs panning.

@evq
Copy link
Member

evq commented Dec 5, 2016

👍

@lifeinchords
Copy link

+(a lot).

@srirambv
Copy link
Collaborator

srirambv commented Feb 8, 2017

+1 from user Jim reported swipe gestures not working on Windows
https://community.brave.com/t/swipe-gesture-not-working/1093

@lucidNTR
Copy link
Contributor

also when three finger swipe is set in osx brave still does two finger swipe navigation, which it should not.

@cndouglas cndouglas added the bug label Mar 18, 2017
@cndouglas cndouglas changed the title Back/Forward Gestures don't work when "Swipe with three fingers" is selected Back/forward gesture does not respect system preferences Mar 18, 2017
@ENDelt260
Copy link

I commented on a similar issue which was closed, and I was directed here. So, just for completeness:
Brave also doesn't respect the settings for a mouse. I use a magic mouse, and have specifically disabled the "Swipe between pages" feature. But, Brave doesn't respect that I've disabled that. This is especially frustrating when using Trello or Google Spreadsheets, or anything else where scroll horizontally, as I find myself inadvertently navigating through my history.

image

@cndouglas
Copy link

Tagging as a serious macOS usability issue. Please re-prioritize if needed.

@thefonso
Copy link

Just adding on here. I'm on MacOS 10.10.5 and Brave 0.13.5 and Brave is not respecting my system preferences of no back/forward swipe between pages gestures.

darkdh added a commit that referenced this issue Mar 26, 2017
Temporarily add system preference swipe setting back until we complete appState migration

fix #7100, #3299

Auditors: @bsclifton, @bbondy

Test Plan:
For swipe conflict
1. Navigate to some sites and make sure you have back/forward history
2. Open dev panel by shift + f8 to make webview smaller
3. Go to https://github.com/brave/browser-laptop/projects/1
4. Swipe left/back should not trigger navigation until reaching the edge

For swipe setting
1. Navigate to some sites and make sure you have back/forward history
2. Turn off "Swipe between edges" in system setting
3. There should be any navigations by edge swiping
4. Turn on "Swipe between edges" in systme setting
5. There will be navigation by edge swiping
@darkdh darkdh modified the milestones: 0.14.0, 0.14.1 Mar 26, 2017
darkdh added a commit that referenced this issue Mar 26, 2017
Temporarily add system preference swipe setting back until we complete appState migration

fix #7100, #3299

Auditors: @bsclifton, @bbondy

Test Plan:
For swipe conflict
1. Navigate to some sites and make sure you have back/forward history
2. Open dev panel by shift + f8 to make webview smaller
3. Go to https://github.com/brave/browser-laptop/projects/1
4. Swipe left/back should not trigger navigation until reaching the edge

For swipe setting
1. Navigate to some sites and make sure you have back/forward history
2. Turn off "Swipe between edges" in system setting
3. There should be any navigations by edge swiping
4. Turn on "Swipe between edges" in systme setting
5. There will be navigation by edge swiping
@cndouglas
Copy link

Thanks @darkdh! I think macOS users will be much happier after these fixes.

With ecd391c, the conflict with horizontal scrolling is fixed. Brave supports two fingers or none at all, depending on the system preferences. Three finger swipe is still not supported: see follow-up issue #7905.

@alexwykoff alexwykoff changed the title Back/forward gesture does not respect system preferences Fixed back/forward gesture to match system preferences Mar 30, 2017
darkdh added a commit that referenced this issue Apr 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.