Update the [Download] tooltip message to [Save] if file is open locally on the machine
Categories
(Firefox :: PDF Viewer, enhancement, P3)
Tracking
()
People
(Reporter: cfogel, Assigned: Snuffleupagus)
References
Details
(Keywords: good-first-bug, Whiteboard: [pdfjs-ux])
Attachments
(1 file, 1 obsolete file)
(deleted),
text/x-github-pull-request
|
Details |
Affected versions
- 81.0b4, 82.0a1(2020-08-30)
Affected platforms
- Windows 10, macOS 10.15, Ubuntu 18;
Steps to reproduce
- Launch Firefox;
- Open any PDF file and save it locally;
- Open the locally saved file;
- Hover over the [Download] icon;
Expected/Enhancement suggestion
- tooltip to show [Save] message;
Actual result
- tooltip shows the [Download] message;
Regression range
- not a regression;
Additional notes
- S4 as suggested severity, since it's a suggestion;
- the reasoning would be to have the tooltip update since the file is already downloaded and saved on the machine.
Updated•4 years ago
|
Updated•4 years ago
|
Updated•3 years ago
|
Comment 1•3 years ago
|
||
Hey, I would like to work on this issue!
Updated•3 years ago
|
Comment 2•3 years ago
|
||
Brendan or Calixte can help you. You can also contact us in the PDF.js room on Matrix: https://chat.mozilla.org/#/room/#pdfjs:mozilla.org.
Comment 3•3 years ago
|
||
:janvi01, you need to get pdf.js and set up a dev environment for it:
https://github.com/mozilla/pdf.js/#getting-the-code
For this bug, you probably want to change the tooltip depending on the file url so you'd need to have a firefox dev environment too:
https://firefox-source-docs.mozilla.org/setup/linux_build.html
You can create the js files for firefox in using gulp mozcentral
from your local pdfjs setup and in moving the generated files to https://searchfox.org/mozilla-central/source/toolkit/components/pdfjs.
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Comment 4•3 years ago
|
||
Comment 5•3 years ago
|
||
This good-first-bug hasn't had any activity for 2 months, it is automatically unassigned.
For more information, please visit auto_nag documentation.
Comment 6•2 years ago
|
||
Romain, do you think we should differentiate the text depending on whether the file is open locally or not, or just always say "Save"?
Updated•2 years ago
|
Comment 7•2 years ago
|
||
Technically the file is always downloaded before the user hit's "Save" - my opinion is this should always be a "Save as..." action.
I pinged content design on https://mozilla-hub.atlassian.net/browse/FFCD-65 to get to a decision
(In reply to Romain Testard [:RT] from comment #7)
Technically the file is always downloaded before the user hit's "Save" - my opinion is this should always be a "Save as..." action.
I pinged content design on https://mozilla-hub.atlassian.net/browse/FFCD-65 to get to a decision
I agree with updating the string to “Save”.
Keeps it consistent with the other hover texts (e.g. “Draw,” “Text,” etc; Makes the action most apparent to the broadest range of users)
Updated•2 years ago
|
Assignee | ||
Comment 9•2 years ago
|
||
Updated•2 years ago
|
Description
•