Page MenuHomePhabricator

Allow users to add "ProfileMenu" items on mobile
ClosedPublic

Authored by epriestley on Mar 27 2019, 9:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 8:51 PM
Unknown Object (File)
Mon, Dec 16, 10:38 AM
Unknown Object (File)
Tue, Dec 3, 10:33 AM
Unknown Object (File)
Mon, Dec 2, 4:51 AM
Unknown Object (File)
Tue, Nov 26, 6:24 AM
Unknown Object (File)
Tue, Nov 26, 6:23 AM
Unknown Object (File)
Tue, Nov 26, 6:23 AM
Unknown Object (File)
Tue, Nov 26, 6:23 AM
Subscribers
None

Details

Summary

Depends on D20337. Fixes T12167. Ref T13272. On this page ("Favorites > Edit Favorites > Personal", for example) the curtain actions aren't available on mobile.

Normally, curtains are built with Controller->newCurtainView(), which sets an ID on the action list, which populates the header button. This curtain is built directly because there's no Controller handy.

To fix the issue, just set an ID. This could probably be cleaner, but that's likely a much more involved change.

Test Plan

Edited my favorites, narrowed the window, saw an "Actions" button.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

src/applications/search/engine/PhabricatorProfileMenuEngine.php
920–921

(This and the other thing below are unused.)

This revision is now accepted and ready to land.Mar 27 2019, 9:34 PM
This revision was automatically updated to reflect the committed changes.