Page MenuHomePhabricator
Differential D11480

Don't activate JX.Scrollbar if we don't need to
ClosedPublic
Actions

Authored by epriestley on Jan 24 2015, 2:18 AM.
Tags
None
Referenced Files
F13139912: D11480.diff
Fri, May 3, 3:22 AM
Unknown Object (File)
Mon, Apr 29, 2:49 PM
Unknown Object (File)
Wed, Apr 24, 10:48 PM
Unknown Object (File)
Wed, Apr 17, 11:23 AM
Unknown Object (File)
Fri, Apr 12, 4:55 AM
Unknown Object (File)
Thu, Apr 11, 7:42 AM
Unknown Object (File)
Wed, Apr 3, 5:39 PM
Unknown Object (File)
Mar 27 2024, 3:39 PM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
chad
Commits
Restricted Diffusion Commit
rP512b17361845: Don't activate JX.Scrollbar if we don't need to
Summary

See discussion on rPc40bc0c8bf75. Fixes a couple of glitchy things:

  • Things were generally not nice on iOS.
  • On OSX, with no mouse, the OS scrollbar and our fake scrollbar would both draw.
  • Bar z-index was not set quite correctly.

Specifically, check if we need these bars. If we don't, just exit immediately and use the OS bars.

Test Plan
  • Tested Safari, Firefox, Chrome with and without a mouse.
  • Tested iOS Simualtor.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 27619.Jan 24 2015, 2:18 AM
epriestley retitled this revision from to Don't activate JX.Scrollbar if we don't need to.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, btrahan.
Herald added a subscriber: epriestley. · View Herald TranscriptJan 24 2015, 2:18 AM
epriestley mentioned this in rPc40bc0c8bf75: Replace the primary scrollbar with a fake one to prepare for a persistent….Jan 24 2015, 2:18 AM
chad accepted this revision.Jan 24 2015, 2:22 AM
chad edited edge metadata.
This revision is now accepted and ready to land.Jan 24 2015, 2:22 AM
Closed by commit rP512b17361845: Don't activate JX.Scrollbar if we don't need to (authored by epriestley, committed by epriestley). · Explain WhyJan 24 2015, 2:22 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
26 lines
webroot/
rsrc/
css/
application/
base/
10 lines
core/
2 lines
externals/
javelin/
lib/
8 lines

Diff 27620

View Options

resources/celerity/map.php

Loading...
View Options

webroot/rsrc/css/application/base/standard-page-view.css

Loading...
View Options

webroot/rsrc/css/core/z-index.css

Loading...
View Options

webroot/rsrc/externals/javelin/lib/Scrollbar.js

Loading...
Phabricator · Built by Phacility