Page MenuHomePhorge

Diffusion: fix missing mobile "Actions" from Manage Repo
ClosedPublic

Authored by valerio.bozzolan on Apr 3 2023, 10:46.
Referenced Files
Unknown Object (File)
Wed, Apr 10, 21:44
Unknown Object (File)
Sun, Apr 7, 23:29
Unknown Object (File)
Sat, Apr 6, 09:43
Unknown Object (File)
Mon, Apr 1, 01:08
Unknown Object (File)
Mon, Apr 1, 01:08
Unknown Object (File)
Mon, Apr 1, 01:08
Unknown Object (File)
Mon, Apr 1, 01:08
Unknown Object (File)
Sat, Mar 30, 09:08

Details

Summary

After this change, the page "/manage/" related to Diffusion
repositories has an "Actions" button for mobile users:

Diffusion after D25109.png (366×670 px, 33 KB)

Before this change, that button was missing only from this
specific page, and so it was really difficult for them
(if not impossible) to edit repository's basic information:

Diffusion before D25109.png (366×670 px, 32 KB)

Do not confuse the hamburger menu on the sidebar with this
"Actions". The hamburger contains queries like Active Repo,
All Repo, etc. Instead the Actions contains
Edit Basic Info, Allow Dangerous Changes, etc.

It seems the cause of this problem is originated from an
apparently incomplete rollback occurred in Sep 6 2017 from
Phabricator:

e1fd74ddb545a2de3963fd60a6c5f4c1d5075814

As you can see, the DiffusionRepositoryBasicsManagementPanel
was probably not completely restored from the previous
version, at least for the part related to the method
newActionList():

bb369c7b711f6eb5f6b3ca9a0c8b85a088b05dc1

The method newActionList() was restored in every other
panel but no the one related to the Basics Management. So,
a new PhabricatorActionListView object was manually
created, but without generating an ID from the Celerity
utilities.

In short, this change assigns an ID to this Curtain, in
order to correctly generates a related "Actions" menu
again. Otherwise, on mobile the curtain (with all the
actions) is automatically hidden, without any alternative.

Closes T15067

Test Plan
  • Repository > Actions > Manage Repository
  • Reduce the page until "Actions" appears (yeeh!)

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Cigaryno added a project: Diffusion (archived).

This change is important.

src/applications/diffusion/management/DiffusionRepositoryBasicsManagementPanel.php
34
This revision is now accepted and ready to land.Apr 6 2023, 07:59