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
F19083640: D20338.id.diff
Tue, Dec 2, 1:07 PM
F19074689: D20338.diff
Mon, Dec 1, 6:21 AM
F19074038: D20338.diff
Mon, Dec 1, 4:03 AM
F19058933: D20338.diff
Sat, Nov 29, 4:00 AM
F18873373: D20338.id48543.diff
Nov 5 2025, 9:24 AM
F18857144: D20338.id.diff
Nov 1 2025, 1:30 PM
F18852073: D20338.diff
Oct 31 2025, 6:39 AM
F18839063: D20338.id.diff
Oct 27 2025, 2:11 PM
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
Branch
panel6
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22405
Build 30658: Run Core Tests
Build 30657: arc lint + arc unit

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.