Page MenuHomePhabricator

When a dropdown menu would render in a way that hides it offscreen, try a different alignment
ClosedPublic

Authored by epriestley on Apr 8 2019, 5:31 PM.
Tags
None
Referenced Files
F17799701: D20383.id48628.diff
Fri, Jul 25, 5:22 AM
F17759745: D20383.id48628.diff
Tue, Jul 22, 6:47 PM
F17747355: D20383.id.diff
Mon, Jul 21, 9:55 PM
Unknown Object (File)
Jun 21 2025, 9:18 PM
Unknown Object (File)
Jun 16 2025, 2:01 PM
Unknown Object (File)
Jun 16 2025, 8:31 AM
Unknown Object (File)
May 31 2025, 9:02 PM
Unknown Object (File)
May 9 2025, 1:48 PM
Subscribers
None

Details

Summary

Depends on D20382. Ref T13272. When something near the edge of the screen has a dropdown menu, we currently may render the menu offscreen.

Instead, keep the menu onscreen.

(This is happening because I'm adding dropdown menus to tab query panels.)

Test Plan

Before:

Screen Shot 2019-04-08 at 10.29.37 AM.png (267×389 px, 22 KB)

After:

Screen Shot 2019-04-08 at 10.29.19 AM.png (279×383 px, 28 KB)

Diff Detail

Repository
rP Phabricator
Branch
portal25
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 22503
Build 30813: Run Core Tests
Build 30812: arc lint + arc unit

Unit TestsFailed

TimeTest
250 msPhabricatorCelerityTestCase::testCelerityMaps
Assertion failed, expected 'true' (at PhabricatorCelerityTestCase.php:32): When this test fails, it means the Celerity resource map is out of date. Run `bin/celerity map` to rebuild it. ACTUAL VALUE
1 msAlmanacNamesTestCase::testServiceOrDeviceNames
30 assertions passed.
0 msAlmanacServiceTypeTestCase::testGetAllServiceTypes
1 assertion passed.
0 msAphrontHTTPSinkTestCase::testHTTPHeaderNames
2 assertions passed.
0 msAphrontHTTPSinkTestCase::testHTTPSinkBasics
3 assertions passed.
View Full Test Results (1 Failed · 356 Passed)