Page MenuHomePhabricator

Add the displayed number to the PagelistWidgetDialog's TopPanel
Closed, ResolvedPublic

Description

The ProofreadPage's pagelist widget contains a panel which displays the current page number type and scan number assigned to the particular page (highlighted in red). We should also add the number currently being higlighted in the pagelist preview (the pagelist preview area is highlighted in green) to the panel.

Screenshot from 2020-09-23 20-08-17.png (679×1 px, 459 KB)

The panel is defined in modules/index.pagelist/PagelistInputWidget.TopPanel.js

Acceptance Criteria

  • The top panel area should have a message saying "Display number: <number>" which should be synced with the label of the highlighted button in the pagelist preview area of the widget.

Installation instructions for the ProofreadPage extension, Installation instructions for the Pagelist Widget

Event Timeline

Change 634827 had a related patch set uploaded (by Diwanshu885; owner: Diwanshu885):
[mediawiki/extensions/ProofreadPage@master] Add the display-number message in Top pannel message list.

https://gerrit.wikimedia.org/r/634827

@Diwanshu885 I've left some feedback on the patch.

@Soda I have updated the code as per your feedback. Please review and check if any further modifications are required.

Change 634827 merged by jenkins-bot:
[mediawiki/extensions/ProofreadPage@master] Add displayed page number to TopPanel

https://gerrit.wikimedia.org/r/634827