[MDL-52290] EditPDF - Also show the total page number in the pagination - Moodle Tracker

XML

Word

Printable

Details

Type: Improvement
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 2.9.3
Fix Version/s: 3.1
Component/s: Assignment
Labels:
- ci
- partner
- patch
- synergy-learning
- ui_change

Affected Branches:
MOODLE_29_STABLE
Fixed Branches:
MOODLE_31_STABLE
Pull from Repository:
git://github.com/davosmith/moodle.git
Pull Master Branch:
~~MDL-52290~~_editpdf_pagecount
Pull Master Diff URL:
https://github.com/davosmith/moodle/compare/moodle:master...MDL-52290_editpdf_pagecount
Testing Instructions:
Hide

(Make sure ghostscript is configured and PDF annotation is working)

As a student submit a PDF to an assignment

As a teacher click on 'grade submission', then 'launch pdf editor'

Check the page number field now reads 'Page 1 of NN' (where NN is the number of pages in the PDF)

Check that next/previous buttons + dropdown select correctly displays 'Page MM of NN' (where MM is the current page number and NN is the total number of pages).
Show
(Make sure ghostscript is configured and PDF annotation is working) As a student submit a PDF to an assignment As a teacher click on 'grade submission', then 'launch pdf editor' Check the page number field now reads 'Page 1 of NN' (where NN is the number of pages in the PDF) Check that next/previous buttons + dropdown select correctly displays 'Page MM of NN' (where MM is the current page number and NN is the total number of pages).

Description

Hi,

status quo is that the pagination starts with Page 1 when opening a PDF with the editor. Thats fine. However if you don't know that behind Page 1 lies a button where you can jump to a specific page and get a clue how many pages the PDF has in total, it is difficult to quickly get to that number.

So I propose that the label could be extended to something like this

Page 1 / 10
Page 1 (10)

So one glance is enough to know how many pages the document has in total.

Attachments

Activity

People

Assignee:: Davo Smith

Reporter:: Kathrin Osswald

Peer reviewer:: Jun Pataleta

Integrator:: Andrew Lyons

Tester:: Adrian Greeve

Participants:: Adrian Greeve, Andrew Lyons, CiBoT, Dan Poltawski, Davo Smith, Jun Pataleta, Kathrin Osswald, Marina Glancy, Mary Cooch

Component watchers:: Adrian Greeve, Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie

Votes:: 1 Vote for this issue

Watchers:: 8 Start watching this issue

Dates

Created:: 25/Nov/15 5:39 AM

Updated:: 20/May/16 2:49 AM

Resolved:: 15/Jan/16 6:57 AM

Fix Release Date:: 23/May/16