Skip to content

Conversation

@calixteman
Copy link
Contributor

Update the styles and HTML to reflect the new views manager concept. For now, nothing about split/merge functionality is implemented or visible. The new styles for the outline, attachments, and layers will be added later.

The thumbnail view is now accessible with the keyboard.

@calixteman calixteman requested a review from a team as a code owner December 9, 2025 21:20
pdfjs-views-manager-status-waiting-ready-label = Getting your file ready…
pdfjs-views-manager-status-waiting-uploading-label = Uploading file…
pdfjs-views-manager-current-outline-item-button =
.title = Find Current Outline Item
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is an outline item?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's an entry in the document outline.
In most of the documents I saw the outline is almost a table of contents but according to the pdf terminology an outline item is sometimes called bookmarks (it's from the specs). In Acrobat Reader the panel dedicated to document outline is labelled "Bookmarks".
Anyway this string was already existing and I just changed its name (see https://searchfox.org/firefox-main/source/toolkit/locales/en-US/toolkit/pdfviewer/viewer.ftl#201).
That said when I see the current translation in french (which is really bad, tbh I don't understand what it means), we need to have some help from the content team about that.

@calixteman calixteman force-pushed the new_sidebar branch 2 times, most recently from 97fee23 to 86b3dbf Compare December 10, 2025 14:42
pdfjs-views-manager-pages-status-waiting-ready-label = Getting your file ready…
pdfjs-views-manager-pages-status-waiting-uploading-label = Uploading file…
## For now keep the previous translations in waiting for some new strings or
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not really useful for localizers, they will translate it as well as they can.

When we change the strings, we will need new IDs. If we only add comments, it will be too late.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok so for now I can just keep the old ids and strings.
And once we've something from the content team, I'll make a follow-up. Does that sound good ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, that's probably the best for localizers.

Update the styles and HTML to reflect the new views manager concept.
For now, nothing about split/merge functionality is implemented or visible.
The new styles for the outline, attachments, and layers will be added later.

The thumbnail view is now accessible with the keyboard.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants