search

hypothesis / client

The Hypothesis web-based annotation client.
Other
641 stars 197 forks source link

Show page numbers on annotation cards #5986

Closed robertknight closed 10 months ago

robertknight commented 11 months ago

We are planning to offer the ability to filter annotations based on page number in VitalSource, and by default annotations outside the current assignment's page range will be hidden. See https://github.com/hypothesis/client/issues/5937.

In order for it to be clearer to the user why they are or are not seeing a particular annotation, it is becoming more important to show the page number on the annotation itself. More generally it would be nice to implement this for PDFs as well, though in order to do that we will need to start capturing page numbers on PDF annotations.

A constraint we will have to handle is that older annotations will not have page numbers available.

robertknight commented 11 months ago

A possible design that displays the page number below the username, where the group name is currently displayed today. I tried displaying the page number on the right side of the quote, as it is closely related information, but there is limited space there. Also there may be a More/Less link to expand or collapse the quote.

I took out the group name to reduce overall volume of information on a card. In the context of the sidebar, that duplicates information available in the toolbar. Outside of the sidebar we might want to display the information in a hierarchy like {group name} - {document info} - {page number}. In the sidebar those first two parts can be elided.

Page number display

The same annotation in the notebook:

Notebook page number
robertknight commented 11 months ago

An alternative to showing the information on the card would be adding additional headings to the thread list. We currently have headings for chapters today, which are only used in VitalSource ebooks. We could add a second level for the page number. The downside is that adding this hierarchy in other contexts (eg. notebook / dashboard) could be complex, so we'd probably still want a way to include the info somewhere on the individual annotation.

robertknight commented 11 months ago

Discussion about a wider release - https://hypothes-is.slack.com/archives/C2C2U40LW/p1702462198924979.

robertknight commented 10 months ago

This is now shipped.