Reduce the size of the close button on infobars and use the correct smaller close icon for tabs
Categories
(Firefox :: Theme, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox113 | --- | fixed |
People
(Reporter: jaws, Assigned: sfoster)
References
(Blocks 3 open bugs)
Details
(Whiteboard: [proton-icons] [proton-infobars] [proton-tabs-bar] [proton-cleanups] [priority:2b][fidefe-Quality-Foundation])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Katie pointed out to me that the close icon for infobars should be reduced in size to use close-12.svg, and the tab's close icon which is already 12px by 12px isn't using close-12.svg, which has better pixel alignment at that size.
While working on this we noticed that the tab close button isn't using the --toolbarbutton-icon-fill color.
Reporter | ||
Comment 1•3 years ago
|
||
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Comment 3•2 years ago
|
||
The severity field for this bug is set to S4
. However, the following bug duplicate has higher severity:
- Bug 1711709: S3
:jaws, could you consider increasing the severity of this bug to S3
?
For more information, please visit auto_nag documentation.
Updated•2 years ago
|
Updated•2 years ago
|
Comment 16•2 years ago
|
||
(In reply to Release mgmt bot [:suhaib / :marco/ :calixte] from comment #3)
The severity field for this bug is set to
S4
. However, the following bug duplicate has higher severity:
- Bug 1711709: S3
:jaws, could you consider increasing the severity of this bug to
S3
?For more information, please visit auto_nag documentation.
S4 seems right for this.
Updated•2 years ago
|
Comment 17•2 years ago
|
||
Comment 18•2 years ago
|
||
Backed out for causing linux build bustage
Backout link: https://hg.mozilla.org/integration/autoland/rev/1b5de4d4f7b727333eb6962dcee9d7725efe3199
Comment 19•2 years ago
|
||
Comment 20•2 years ago
|
||
Backed out by mistake.
Relanded the changes, sorry
Comment 21•2 years ago
|
||
bugherder |
Updated•2 years ago
|
Comment 22•2 years ago
|
||
Backed out changeset 17805375f92a (Bug 1710910) for causing Bug 1818233.
Backout link: https://hg.mozilla.org/integration/autoland/rev/65df162e540fcc392bd0f8fb020abbfc7f60ecee
Comment 23•2 years ago
|
||
Backout merged to central: https://hg.mozilla.org/mozilla-central/rev/65df162e540f
Comment 24•2 years ago
|
||
I'll look into this. Thanks for flagging!
Comment 25•2 years ago
|
||
There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:prathiksha, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit auto_nag documentation.
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Assignee | ||
Comment 26•2 years ago
|
||
I'll see if I can gets this re-landed.
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Comment 27•2 years ago
|
||
Comment 28•2 years ago
|
||
bugherder |
Description
•