Closed Bug 1389755 Opened 7 years ago Closed 7 years ago

"Show more bookmarks" button should not use the same icon as "More tools..." button and could be narrower

Categories

(Firefox :: Theme, defect)

57 Branch
x86_64
Windows 7
defect
Not set
normal

Tracking

()

VERIFIED WONTFIX

People

(Reporter: Virtual, Unassigned)

Details

(Keywords: nightly-community)

Attachments

(3 files)

STR:
1. Add some item to "More tools..." menu to see "More tools..." button in URL/address/location bar
2. Add some bookmarks to see "Show more bookmarks..." in Bookmarks Toolbar
and see that they're the same compared to before

It's also bigger, which partially fixes bug #1368580,
but I think "Show more bookmarks..." and "More tools..." buttons could be 1/2 in width compared to other buttons.

Caused by:
Bug #1377184 - Use new Photon hover effect also on "Show more bookmarks" and extensions buttons icons in Bookmarks Toolbar
Has Regression Range: --- → yes
Has STR: --- → yes
Attachment #8896572 - Attachment description: before landing patch from Bug #1368580.png → before landing patch from Bug #1377184.png
Attachment #8896572 - Attachment filename: before landing patch from Bug #1368580.png → before landing patch from Bug #1377184.png
Attachment #8896573 - Attachment description: after landing patch from Bug #1368580 - 1.png → after landing patch from Bug #1377184 - 1.png
Attachment #8896573 - Attachment filename: after landing patch from Bug #1368580 - 1.png → after landing patch from Bug #1377184 - 1.png
Attachment #8896574 - Attachment description: after landing patch from Bug #1368580 - 2.png → after landing patch from Bug #1377184 - 2.png
Attachment #8896574 - Attachment filename: after landing patch from Bug #1368580 - 2.png → after landing patch from Bug #1377184 - 2.png
This is a generic chevron icon that can be used in multiple places.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Whiteboard: [photon-visual] [triage]
OK. Thank you very much for explanation.
No longer blocks: 1377184
Status: RESOLVED → VERIFIED
Has Regression Range: yes → ---
Has STR: yes → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: