microsoft / vscode

Visual Studio Code
https://code.visualstudio.com
MIT License
164.94k stars 29.52k forks source link

Panel badge should adopt style of activity bar #195998

Open bpasero opened 1 year ago

bpasero commented 1 year ago

They now look quite different:

Image

I would suggest a consistent style and color for all.

sandy081 commented 1 year ago

Got color, but can you elaborate on style?

bpasero commented 1 year ago

Maybe just the positioning, it seems quite far away from the label. I guess we cannot really do the overlay...

I wonder why we never went with icons for the panel? or maybe icon and text and the badge could be over the icon?

bpasero commented 1 year ago

I think a wrongly referenced commit message.

sandy081 commented 1 year ago

I wonder why we never went with icons for the panel? or maybe icon and text and the badge could be over the icon?

Yeah, this is a very good thought

CC @hbons @daviddossett

daviddossett commented 1 year ago

I think this would be partly due to the difference between activityBarBadge.background and badge.background in our default themes. Aligning them could help here—I was already thinking of reverting back to that anyway.

Maybe just the positioning, it seems quite far away from the label.

Any specific reasoning for the gap between text and badge in panel tabs @Tyriar?

Tyriar commented 1 year ago

@daviddossett I think it's just been like that forever, before my time 🙂

moritztim commented 11 months ago

I would love to have Icons in the panel! Here's a mockup of the color scheme (Dark+ theme): I think a border would be needed between the side bars and the panel Mockup In comparison to the current color scheme, the mockup is definitely an improvement, imo. Original

moritztim commented 11 months ago

Mistakenly referenced in f1b7a36601d8d0508704f3402ee9376128afcbcc

joaomoreno commented 3 months ago

These will be aligned with https://github.com/microsoft/vscode/issues/227076

gjsjohnmurray commented 2 months ago

These will be aligned with #227076

This has brought us the option for icons instead of labels 👍, but the badge appearance hasn't been aligned with that of the side bars.

sandy081 commented 2 months ago

That's correct and I can confirm

Image