Page MenuHomePhabricator

Provide a scrollbar when properties overflow on desktop
ClosedPublic

Authored by chad on Oct 12 2014, 9:15 PM.
Tags
None
Referenced Files
F14075947: D10691.id25664.diff
Thu, Nov 21, 1:45 PM
Unknown Object (File)
Wed, Nov 20, 8:36 AM
Unknown Object (File)
Sat, Nov 16, 9:11 AM
Unknown Object (File)
Tue, Nov 12, 2:46 AM
Unknown Object (File)
Thu, Nov 7, 8:47 PM
Unknown Object (File)
Thu, Oct 24, 7:00 AM
Unknown Object (File)
Oct 22 2024, 5:32 PM
Unknown Object (File)
Oct 21 2024, 6:57 AM

Details

Summary

Fixes issues seen in D10690 with unit results.

Test Plan

test D10690 and locally

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Provide a scrollbar when properties overflow on desktop.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.

Doesn't scroll mean there's always a scrollbar? Is that definitely right? (Not auto?)

Maybe Chrome changed behaviour, but I think you're right.

chad edited edge metadata.

Use auto

If you have OSX and a laptop, and do not have a USB mouse connected, the OS doesn't really render normal scrollbars (in pretty much any app). Plugging in a mouse gives you scrollbars. Might be the issue?

"Mouse"? Is that some 90's technology?

e.g., page with a USB mouse connected:

Screen_Shot_2014-10-12_at_2.41.50_PM.png (933×1 px, 138 KB)

epriestley edited edge metadata.
This revision is now accepted and ready to land.Oct 12 2014, 9:42 PM

It's a video game controller that helps MLG pros like myself get headshots and retain their Mountain Dew sponsorships.

This revision was automatically updated to reflect the committed changes.