-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Change the sidebar for a views manager #20495
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
l10n/en-US/viewer.ftl
Outdated
| 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 |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
97fee23 to
86b3dbf
Compare
l10n/en-US/viewer.ftl
Outdated
| 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 |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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 ?
There was a problem hiding this comment.
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.
86b3dbf to
5b231f7
Compare
5b231f7 to
e250fad
Compare
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.
e250fad to
e023ab9
Compare
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.