Browser-laptop: Fixed back/forward gesture to match system preferences

Created on 22 Aug 2016  路  9Comments  路  Source: brave/browser-laptop

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.

  • Platform:
    macOS 10.11.6
  • Brave Version:
    0.11.5
  • Steps to reproduce:
    i. select "Swipe with three fingers" in Tackpad Settings->More Gestures->Swipe between pages
    ii. try to use three finger back/forward swipe gesture in brave
  • Related Issue:
    Support MacBook trackpad pinch to zoom #1364
    Two-finger swipe gesture on macOS should have visual indicator #2477
    Support for three finger tap: "Look up & data detectors"
    Simulated 3 Finger Swipe Gestures not working #6587
OmacOS Qchecked-macOS Qtest-plan-specified bug release-noteinclude

Most helpful comment

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

All 9 comments

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

:+1:

+(a lot).

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

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

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

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

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.

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.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

shortstuffsushi picture shortstuffsushi  路  3Comments

jonathansampson picture jonathansampson  路  3Comments

bbondy picture bbondy  路  3Comments

jonathansampson picture jonathansampson  路  3Comments

bbondy picture bbondy  路  3Comments