Bug 1699631 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
toolkit/themes/shared/icons/icon-refresh.svg (this one was taken from the fxr directory for the restart to update thing, it should have taken the reload.svg from browser instead).
browser/themes/shared/icons/menus/reload.svg (this one was introduced along with the new Windows context menu style, and should be irrelevant with the new proton icon drop)
browser/themes/shared/icons/reload.svg

DevTools also have duplicate icons (but could be solved separately):
devtools/client/debugger/images/reload.svg
devtools/client/themes/images/reload.svg

Suggested plan:
* icon-refresh.svg moved back to browser/fxr/content/assets/
* reload.svg moved to toolkit/themes/shared/icons
* remove browser/themes/shared/icons/menus/reload.svg 
* make everything except devtools & fxr use the toolkit icon
Revision 1 by
on 
* toolkit/themes/shared/icons/icon-refresh.svg 
this one was taken from the fxr directory for the restart to update thing, it should have taken the reload.svg from browser instead
* browser/themes/shared/icons/menus/reload.svg
this one was introduced along with the new Windows context menu style, and should be irrelevant with the new proton icon drop
* browser/themes/shared/icons/reload.svg

DevTools also have duplicate icons (but could be solved separately):
* devtools/client/debugger/images/reload.svg
* devtools/client/themes/images/reload.svg

Suggested plan:
* icon-refresh.svg moved back to browser/fxr/content/assets/
* reload.svg moved to toolkit/themes/shared/icons
* remove browser/themes/shared/icons/menus/reload.svg 
* make everything except devtools & fxr use the toolkit icon

Back to Bug 1699631 Comment 0