← Back to team overview

ubuntu-webapps-bugs team mailing list archive

[Bug 1426567] [NEW] Support pinch-viewport and overlay scrollbars on desktop

 

Public bug reported:

A change I'm about to push in order to finally get correctly behaving
scrollbars on the device breaks overlay scrollbars and pinch-viewport on
desktop, so I've reported this to do extra work later on to make them
both work.

- The overlay scrollbar thumb style is defined in
third_party/WebKit/Source/platform/scroll/ScrollbarThemeAura.cpp to be
10DIPs wide, hit-tested and no margin. I've modified this to be 3DIPs
wide with a 3DIP margin and no hit-testing, which is great for the
device but sucks on desktop.

- I've modified third_party/WebKit/Source/core/frame/FrameView.cpp to
disable scrollbars for the main frame when pinch-viewport is enabled, as
we display the pinch-viewport scrollbars all the time because these are
the ones that stick to the device viewport (the same as Chrome on
Android). Chrome on the desktop still displays non-overlay scrollbars
attached to the outer viewport for the main-frame, and then displays the
pinch-viewport overlay scrollbars (attached to the device viewport)
during pinch, but Oxide would need additional work to support both of
these.

** Affects: oxide
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Ubuntu
WebApps bug tracking, which is subscribed to Oxide.
https://bugs.launchpad.net/bugs/1426567

Title:
  Support pinch-viewport and overlay scrollbars on desktop

Status in Oxide Webview:
  New

Bug description:
  A change I'm about to push in order to finally get correctly behaving
  scrollbars on the device breaks overlay scrollbars and pinch-viewport
  on desktop, so I've reported this to do extra work later on to make
  them both work.

  - The overlay scrollbar thumb style is defined in
  third_party/WebKit/Source/platform/scroll/ScrollbarThemeAura.cpp to be
  10DIPs wide, hit-tested and no margin. I've modified this to be 3DIPs
  wide with a 3DIP margin and no hit-testing, which is great for the
  device but sucks on desktop.

  - I've modified third_party/WebKit/Source/core/frame/FrameView.cpp to
  disable scrollbars for the main frame when pinch-viewport is enabled,
  as we display the pinch-viewport scrollbars all the time because these
  are the ones that stick to the device viewport (the same as Chrome on
  Android). Chrome on the desktop still displays non-overlay scrollbars
  attached to the outer viewport for the main-frame, and then displays
  the pinch-viewport overlay scrollbars (attached to the device
  viewport) during pinch, but Oxide would need additional work to
  support both of these.

To manage notifications about this bug go to:
https://bugs.launchpad.net/oxide/+bug/1426567/+subscriptions


Follow ups

References