Page MenuHomePhabricator

Hide the Differential scroll objective list on trackpad systems
ClosedPublic

Authored by epriestley on May 20 2017, 11:15 AM.
Tags
None
Referenced Files
F19798600: D17974.diff
Fri, Feb 27, 9:35 PM
F19685404: D17974.diff
Mon, Feb 9, 8:34 AM
F19150870: D17974.diff
Dec 10 2025, 9:46 PM
F19097313: D17974.id.diff
Dec 4 2025, 1:00 PM
F19090216: D17974.diff
Dec 3 2025, 10:56 AM
F19051906: D17974.id43231.diff
Nov 28 2025, 2:24 AM
F19039795: D17974.id43231.diff
Nov 26 2025, 9:47 AM
F18865618: D17974.id.diff
Nov 3 2025, 1:09 PM
Subscribers
None

Details

Summary

Ref T12733. In the longer run I'd like to just push this out from the edge, but that currently gets us into trouble since we start bumping into content. On my system, the trackpad scrollbar also expands in size when moused over, so the minimum number of pixels we need to push it out is approximatley 15px. This hits body content and the persistent chat.

For now, just disable this element on trackpad systems.

Test Plan

Disconnected all USB peripherals, quit and relaunched Safari, saw no objective list.

Reconnected mouse, relaunched Safari, saw objective list.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.May 20 2017, 1:51 PM
This revision was automatically updated to reflect the committed changes.