terminal icon indicating copy to clipboard operation
terminal copied to clipboard

open new tab icon at the top title is not centered correctly

Open Emirhan-7274 opened this issue 1 year ago • 1 comments

Windows Terminal version

1.19.10821.0

Windows build number

10.0.22631.3447

Other Software

No response

Steps to reproduce

Ekran görüntüsü 2024-04-20 151314

its hard to notice it but if you set "Use acrylic material in the tab row" to true it becomes highly noticable

Expected Behavior

open a new tab icon to be centered

Actual Behavior

open a new tab icon is not centered

Emirhan-7274 avatar Apr 20 '24 12:04 Emirhan-7274

Huh, good eye.

Code is somewhere around here:

https://github.com/microsoft/terminal/blob/99061ee272b72c6d565802068f7b09ab12e4173e/src/cascadia/TerminalApp/TabRowControl.xaml#L48-L70

IIRC we're adding some margin to make the + centered nicely, but that code hasn't been touched in years so I may be off base on that

zadjii-msft avatar Apr 23 '24 11:04 zadjii-msft

I think the SplitButton component doesn't support customizing the spacing for the secondary button: https://github.com/microsoft/microsoft-ui-xaml/issues/5605.

Currently, it has the padding hardcoded: https://github.com/microsoft/microsoft-ui-xaml/blob/7180ad3d3998bcbf2c8fa028ffb1ce3049d78228/dev/SplitButton/SplitButton.xaml#L297. And the icon always aligned to the right.

When the button has the default secondary button size, it renders the icon centered properly.

image

For other sizes, it renders wrong.

image

Would using the default size be ok?

orlandoisay avatar Jul 04 '24 05:07 orlandoisay