Page MenuHomePhorge

Dashboard Panel: fix first tab sometime not opened anymore
ClosedPublic

Authored by valerio.bozzolan on Oct 12 2023, 13:37.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 13:48
Unknown Object (File)
Wed, Apr 24, 08:49
Unknown Object (File)
Wed, Apr 24, 08:49
Unknown Object (File)
Tue, Apr 9, 03:46
Unknown Object (File)
Sun, Apr 7, 10:29
Unknown Object (File)
Thu, Apr 4, 18:08
Unknown Object (File)
Mon, Apr 1, 19:24
Unknown Object (File)
Mon, Apr 1, 02:55

Details

Summary

After this change the first Tab ID is always selected, so we don't risk
to have no tabs opened by default under some corner cases.

This is vaguely better than the original logic that relied on the fact that
(0=="0") is true. Besides being a vaguely weird emoticon as well, now that
I look at it better.

Original logic:

https://we.phorge.it/source/phorge/browse/master/src/applications/dashboard/paneltype/PhabricatorDashboardTabsPanelType.php;c43618a3a8bb021936fad687f04cb1a95faa23e4$289

This fixes a regression that appeared in the cute Wikimedia Phabricator
(that now is really Wikimedia Phorge). Their homepage should be gorgeous again,
without blank space due to unopened silly tabs.

Closes T15651

Test Plan
  • Have a Tab Panel (/dashboard/panel/).
  • Be sure that the first Tab is automatically selected (again?)

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

add useful comment (at least in my opinion) about $idx

This revision is now accepted and ready to land.Oct 16 2023, 06:19