Page MenuHomePhabricator

Conpherence - override some device specific styles so menu dropdown always present
ClosedPublic

Authored by btrahan on May 18 2015, 9:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 17, 4:10 AM
Unknown Object (File)
Mon, Dec 9, 2:26 PM
Unknown Object (File)
Fri, Dec 6, 6:00 AM
Unknown Object (File)
Thu, Dec 5, 2:18 AM
Unknown Object (File)
Wed, Dec 4, 2:59 AM
Unknown Object (File)
Wed, Dec 4, 2:59 AM
Unknown Object (File)
Wed, Dec 4, 2:59 AM
Unknown Object (File)
Wed, Dec 4, 2:59 AM
Subscribers

Details

Summary

Fixes T7684. I am not sure if this is simple or not in CSS terms but I just blasted a bunch of css classes to make sure they have the proper "device-desktop" values for the conpherence durable column.

One small visual oddity is the main menu dropdowns (for conpherence and the actual page) should have a hover state css, but we add device-tablet too early relative to the actual page width. This is no big deal probably.

Test Plan

made the screen table sized. used the dropdown menu with great success!

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Conpherence - override some device specific styles so menu dropdown always present.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added reviewers: epriestley, chad.

Touch devices don't do hover states so we restrict them to device-desktop. Imperfect though since we relate screen size to device when that might not be true in the durable column case.

btrahan edited edge metadata.

have hover styles behave consistently

epriestley edited edge metadata.
This revision is now accepted and ready to land.May 19 2015, 1:38 PM
This revision was automatically updated to reflect the committed changes.