Last modified: 2013-04-17 10:43:59 UTC

Wikimedia Bugzilla is closed!

Wikimedia has migrated from Bugzilla to Phabricator. Bug reports should be created and updated in Wikimedia Phabricator instead. Please create an account in Phabricator and add your Bugzilla email address to it.
Wikimedia Bugzilla is read-only. If you try to edit or create any bug report in Bugzilla you will be shown an intentional error message.
In order to access the Phabricator task corresponding to a Bugzilla report, just remove "static-" from its URL.
You could still run searches in Bugzilla or access your list of votes but bug reports will obviously not be up-to-date in Bugzilla.
Bug 28868 - display page count (of the multi-paged media file) in dimensions
display page count (of the multi-paged media file) in dimensions
Status: NEW
Product: MediaWiki
Classification: Unclassified
DjVu (Other open bugs)
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
: design
Depends on: 28881
  Show dependency treegraph
Reported: 2011-05-07 09:30 UTC by John Mark Vandenberg
Modified: 2013-04-17 10:43 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---

patch to add "Current page X" (3.46 KB, patch)
2013-04-17 00:56 UTC, Bawolff (Brian Wolff)
screenshot of proposed change to multipage layout (275.95 KB, image/png)
2013-04-17 00:59 UTC, Bawolff (Brian Wolff)

Description John Mark Vandenberg 2011-05-07 09:30:57 UTC
Media handlers can change the content within the dimension field displayed underneath the page.
It would be useful to include page count in this field.
Comment 1 Bawolff (Brian Wolff) 2011-05-07 19:49:39 UTC
Fixed in r87651.

Note that revision won't affect PagedTiffHandler, which does other things with the long description, including showing which page you're currently viewing. I'm unsure if it would look stupid to have:

(page 1, 3,055 × 4,108 pixels, file size: 35.94 MB, MIME type: image/tiff, 42 pages)

Maybe something like:
(page 1, 3,055 × 4,108 pixels, file size: 35.94 MB, MIME type: image/tiff, 42 pages total)

Would look better? Thoughts?

(leaving this bug open for the moment to comment on the tiff handler)
Comment 2 John Mark Vandenberg 2011-05-08 08:55:07 UTC
I don't think the dimensions field should change depending on the page you are viewing.  The page being viewed is not a dimension.  The dimensions are listed underneath the current image, *however* that is in round brackets after "Full resolution‎".  When the user clicks on "Full resolution", they don't download only 'page 2' when they are viewing page two - the entire document is downloaded.

The tiff viewer is displaying 'page 1' for single page images (e.g. [[File:D.tiff]]
And I can't find a multipage tiff on commons so I uploaded one
[[File:International Convention for Regulation of Whaling.tiff]]
Comment 3 John Mark Vandenberg 2011-05-08 09:01:25 UTC
Did this code apply to djvus and pdfs? (is bug 28869 able to be closed?)
Comment 4 Bawolff (Brian Wolff) 2011-05-08 09:29:29 UTC
It should apply to PDFHandler extension, DjVu files, and Tiff files if using the built in (Crappy) tiff handler.

The only one it doesn't apply to is tiffs when the wiki is using PagedTiffHandler extension (As opposed to built in tiff support). All Wikimedia wikis use PagedTiffHandler.

The change to make this work for PagedTiffHandler is trivial, just per what I said in comment 1 I'm a little worried it might look stupid ;)
Comment 5 Bawolff (Brian Wolff) 2011-05-09 02:18:04 UTC
btw, cc'ing Markus Glaser because he is default assignee for PagedTiffHandler, and I've somewhat hijacked this bug to be something about PagedTiffHandler.
Comment 6 Markus Glaser 2011-05-13 16:39:12 UTC
@Bawolff, thanks for cc'ing!
In r88006, I removed the page number in longdesc for PagedTiffHandler and put in the total count of pages, which should fix this bug.

However, there are things to consider:
* I know of multipage tiffs where height and width vary over the pages. At the moment, in the short description, the resolution of the first page is used, and in long description, the resolution of the currently shown page is used. I couldn't think of a better solution, so if you have any thoughts...
* There is no real indicator any more on which page you are when viewing a multipage image. There is the correct page number next to "Go to page", but, obviously, the label says something different. I suggest we put a text above, saying "Current page" with the page number and set the number next to "go to page" to the next page.

What do you think?
Comment 7 Bawolff (Brian Wolff) 2013-04-17 00:32:18 UTC
Wow, I guess I'm not good at following up on bugs...

Yes I agree, changing it to be

Currently displaying Page X
Goto page <next page>

Would make sense
Comment 8 Bawolff (Brian Wolff) 2013-04-17 00:56:09 UTC
Created attachment 12121 [details]
patch to add "Current page X"

Hmm, tried it. Not sure if I 100% like the results. Maybe could use some feedback from commons (or UX people?).

Comments? I'll also attach a screenie
Comment 9 Bawolff (Brian Wolff) 2013-04-17 00:59:06 UTC
Created attachment 12122 [details]
screenshot of proposed change to multipage layout

Screenie of proposed changes. Please comment/suggest tweaks
Comment 10 Markus Glaser 2013-04-17 10:43:59 UTC
I had a similar idea when working on the PagedTiffHandler, but never came to implement it. So a big +1. From a usability perspective, it might be a good idea to have the current page number highlighted somehow. If we use a form element to switch to the next page, that's going to attach readers attention and they might think that's the current page number.


Note You need to log in before you can comment on or make changes to this bug.