HomePhabricator

Keep related changes list visible, if it contains visible elements

Authored by QChris on Jul 4 2020, 9:52 PM.

Description

Keep related changes list visible, if it contains visible elements

The related changes list used to hide itself, if none of the built-in
change lists contained elements. This rule ignored plugins. So if all
built-in change lists were empty, but a plugin (e.g.: zuul plugin)
injected content, the plugin content was invisible.

We now hide the related changes list only if we can assert that all
children are invisible. Thereby plugin content stays visible, even if
all built-in change lists are empty.

Change-Id: Ifc941fa479ed25adb328f07f4f2a088db57d8be0

Details

Committed
QChrisJul 18 2020, 8:59 AM
Parents
rGERRITDEPLOY2b8c5b8aa6b7: Fix the position of the hovercard
Branches
Unknown
Tags
Unknown
ChangeId
Ifc941fa479ed25adb328f07f4f2a088db57d8be0