The "Save file" icon is inconsistent on browser toolbar & pdf.js toolbar #926
Labels
Class::Icon
Icon for panels, menus, libraries
Component::UserContents
Page related issues except Activity-Stream, not browser UI
Issue::Bug
Something isn't working
Describe the bug
The "Save File" icon (toolbarButton-download.svg) on the browser toolbar & pdf.js toolbar is inconsistent.
I know, that this very little detail is almost unobservable. Still, I believe, that the "Save File" icon on Firefox PDF reader (pdf.js) is using an older pre-Lepton icons redesign variant with a thicker arrow head.
Here is a comparison (the upper is the browser toolbar and the bottom is the pdf.js toolbar):
![Inconsistent toolbarButton-download icons](https://private-user-images.githubusercontent.com/69351811/340803802-bee431c5-c3c0-4a84-96b4-6c128f57b59f.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjExMzcxNTEsIm5iZiI6MTcyMTEzNjg1MSwicGF0aCI6Ii82OTM1MTgxMS8zNDA4MDM4MDItYmVlNDMxYzUtYzNjMC00YTg0LTk2YjQtNmMxMjhmNTdiNTlmLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MTYlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzE2VDEzMzQxMVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWE1NDBjNzM5NWIzZDJmMzc1Y2RhZTQzNjZjODNjM2NhZTEwZDQ1YzI4NzJhMGY0MTA1YzkxNWQ5MDMyZGIyODMmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0._7LAlRGk2ksffG1IB95uvP7NVQJevf4y-JVYONoHDqQ)
Expected behavior
My solution: Use the newer variant (with a thinner arrow head) from the browser toolbar also to the Pdf.js toolbar.
To avoid misunderstanding:
(By the way, I remember, that changing the "Save File" icon on Pdf.js was my first proposal ever on this project and I didn't expect, that I would go back to this change today).
Screenshots
No response
OS
Windows 10
OS - Others
No response
Firefox Version
v127.0.0 (64-bit)
Distribution
Theme
Theme - More Info
No response
user.js
setupuser.js
setupAdditional context
No response
The text was updated successfully, but these errors were encountered: