This sidebar is no longer a "proof of concept". Rename to "sidebar", perhaps?
Any conditionals like if ( this.pocSidebar ) can be rewritten, this is always true now.
Try to clean up this issue in MWTemplateDialog:
// FIXME: This is created at the wrong time. That's why we run into the situation // where an old instance exists. Should be in initialize().
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T289653 Screenreader support for the new VisualEditor template editor sidebar | |||
| Resolved | thiemowmde | T290262 Add keyboard shortcuts for VE template dialog | |||
| Open | None | T294521 Finishing touches on keyboard shortcuts in the new template dialog | |||
| Resolved | thiemowmde | T299490 Add shortcuts to hover tooltips on buttons in adv toolbar | |||
| Resolved | thiemowmde | T289043 [Epic] Synchronize scroll, highlight, and focus in new VE template dialog sidebar and content pane | |||
| Resolved | thiemowmde | T290975 Synchronize scroll and focus when template dialog is in narrow-screen mode | |||
| Resolved | Lena_WMDE | T299036 VE Dialog: Clicking template or parameter name scrolls it to top of dialog | |||
| Resolved | awight | T289049 Disconnect old sidebar from VE template dialog | |||
| Resolved | thiemowmde | T310868 Rename template dialog "pocSidebar" |
Change 806910 had a related patch set uploaded (by Svantje Lilienthal; author: Svantje Lilienthal):
[mediawiki/extensions/VisualEditor@master] Rename template dialog pocSidebar
Change 806910 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Rename template dialog pocSidebar
Change 806938 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):
[mediawiki/extensions/VisualEditor@master] Remove some "pocSidebar" related pieces from template dialog
Change 806938 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Remove some "pocSidebar" related pieces from template dialog