Skip to content

Conversation

@jiawei-hong
Copy link
Contributor

@jiawei-hong jiawei-hong commented Nov 7, 2023

fix #197648

before

before

after

after

@gjsjohnmurray
Copy link
Contributor

Does this affect how badges look? One way to check is to downgrade one of your extensions, then check for updates. The Extensions activity icon should display a number badge.

@gjsjohnmurray
Copy link
Contributor

pinging @sandy081

@jiawei-hong
Copy link
Contributor Author

jiawei-hong commented Nov 7, 2023

@gjsjohnmurray I don't think this change will affect the badge, Below is my test under the dark modern theme and high contrast theme

Dark Modern Theme

Screenshot 2023-11-08 at 00 23 22

Dark High Contrast theme

Screenshot 2023-11-08 at 00 23 32

@jiawei-hong jiawei-hong closed this Nov 7, 2023
@jiawei-hong jiawei-hong force-pushed the style/activity-bar-icon-size branch from 0d86eec to c5203fe Compare November 7, 2023 16:33
@jiawei-hong jiawei-hong reopened this Nov 7, 2023
@aeschli aeschli assigned sandy081 and unassigned aeschli Nov 7, 2023
@sandy081
Copy link
Member

sandy081 commented Nov 9, 2023

Activity bar icon size is made bigger for a reason. So, I would not like to remove that. If outline border is the problem, should that be fixed?

@jiawei-hong
Copy link
Contributor Author

Activity bar icon size is made bigger for a reason. So, I would not like to remove that. If outline border is the problem, should that be fixed?

I don't think the outline border is the problem; the real problem is that the ::before element width exceeds the a tag width
We can directly set the a tag width to be equal to the font-size of the ::before element

Will new problem or concern if I set a tag width is equal to the font-size of the ::before element

@sandy081
Copy link
Member

Will new problem or concern if I set a tag width is equal to the font-size of the ::before element

Should not be. Please check if the outline and badges are looking fine after the change.

@sandy081 sandy081 closed this Jan 16, 2024
@microsoft microsoft locked and limited conversation to collaborators Jun 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

style: activiy icon size is weird

4 participants