| STran | |
| Dec 7 2020, 7:45 PM |
| F33942079: image.png | |
| Dec 10 2020, 7:00 PM |
| F33942077: image.png | |
| Dec 10 2020, 7:00 PM |
| F33940059: image.png | |
| Dec 8 2020, 8:21 PM |
| F33940061: image.png | |
| Dec 8 2020, 8:21 PM |
Make the panel implemented in T268297 collapsible like the search form on the same page
Requirements:
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Make the infobox collapsible | mediawiki/extensions/IPInfo | master | +43 -17 |
@STran Hat tip - you can just mention the ticket number to link to it like T268297 will appear like T268297. And if you add curly braces around it, it expands to the ticket title - {T268297} becomes T268297: Move over existing IP Information from popup to the contributions page.
Change 647050 had a related patch set uploaded (by STran; owner: STran):
[mediawiki/extensions/IPInfo@master] Make the infobox collapsible
@Prtksxna As mentioned in the standup today, here's the discrepancy in appearance between our infobox and the collapsible form:
With focus:
We could fix it by copying exactly what the collapsible form does, but I wouldn't recommend it... The collapsible form uses a CollapsibleFieldsetLayout, a custom PHP class just for the form use-case - a lot of the CSS relies on there being a form. This isn't really re-usable, and we'd have to essentially re-implement it in JS, then copy the CSS without requiring a form. It'll be difficult to understand and maintain, and to keep up to date if changes are made to the collapsible form.
In @STran's patch, the infobox just uses a button, which is a lot more straightforward to understand.
We do want them to look the same eventually, and I think the safest way to do this will be by creating some collapsible panel component that will work for the form and for our infobox, and can be reused anywhere: T269878
Change 647050 merged by jenkins-bot:
[mediawiki/extensions/IPInfo@master] Make the infobox collapsible
@Tchanders this looks good. I think more important than the padding is the total click area of the button (which is larger in the current box). Can the button be made wider without any of the collapsible panel component work that you've planned.
