mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor
synced 2024-11-15 18:39:52 +00:00
74bffc5d0d
1. Before, removePages() was calling setPage() with null. This makes sense for removed top-level parts because these are really removed from both sides of the dialog. Template parameters are never removed. Only unchecked (and removed from the right side of the dialog, but this is not what this code is about). When I navigate a parameter list and uncheck a parameter I need the focus and highlighting to stay. 2. We have a dedicated method when a parameter is unchecked. This can check if the removed parameter is also the selected one (called "set" in this code) and can reset this state. Without losing the highlighting or anything else. Bug: T312213 Change-Id: Ibb717ca49cae805617ebee196937c79daa72f1c1 |
||
---|---|---|
.. | ||
ve.ui.MWTwoPaneTransclusionDialogLayout.js | ||
ve.ui.MWVerticalLayout.js |