Page MenuHomePhabricator

hide-if and flatlist do not cooperate well
Closed, ResolvedPublic

Assigned To
Authored By
matej_suchanek
Apr 6 2018, 7:15 PM
Referenced Files
F16764346: obrazek.png
Apr 6 2018, 7:15 PM
F16764327: obrazek.png
Apr 6 2018, 7:15 PM
Tokens
"Like" token, awarded by Daimona.

Description

In https://gerrit.wikimedia.org/r/#/c/424245/ and https://gerrit.wikimedia.org/r/#/c/424652/ I want to make use of [[ https://phabricator.wikimedia.org/source/mediawiki/browse/master/includes/htmlform/HTMLForm.php;48a6894abbb26e5e93437bf30cf0fee54db052a4$94 | hide-if ]] setting to conditional show/hide some form elements.

The element to be hidden (highlighted in the screenshots) is 'type' => 'radio' with 'flatlist' => true. The screenshots show that the look is different with and without hide-if (if it's used, flatlist is ignored).

without hide-ifwith hide-if
obrazek.png (545×919 px, 14 KB)
obrazek.png (623×919 px, 15 KB)

Event Timeline

I investigated a bit. The problem comes from the infusion performed here. In fact, as the element is infused it loses the flatlist property. I still have to determine the specific reason, though.

Daimona claimed this task.

This is now working as expected, likely fixed independently by someone.