Homestyx hydra
Diffusion hydra 524579fe64a6

Diffusion: fix missing mobile "Actions" from Manage Repo

Description

Diffusion: fix missing mobile "Actions" from Manage Repo

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

{F276113}

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:

{F276115}

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!)

Reviewers: O1 Blessed Committers, Cigaryno, avivey

Reviewed By: O1 Blessed Committers, Cigaryno, avivey

Subscribers: speck, tobiaswiese, Matthew, Cigaryno

Tags: #diffusion, #mobile

Maniphest Tasks: T15067

Differential Revision: https://we.phorge.it/D25109

Details

Provenance
Valerio BozzolanAuthored on Apr 6 2023, 4:58 AM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:787a84969fa0: Phriction: clarify its search results as "Wiki page"
Branches
Unknown
Tags
Unknown

Event Timeline