Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Icon on button is misaligned #837

Closed
vopani opened this issue May 25, 2021 · 5 comments
Closed

Icon on button is misaligned #837

vopani opened this issue May 25, 2021 · 5 comments
Assignees
Labels
bug Bug in code ui Related to UI

Comments

@vopani
Copy link
Contributor

vopani commented May 25, 2021

Wave SDK Version, OS

v0.16.0 in Chrome on Mac

Actual behaviour

Icons in buttons are not vertically centre-aligned and looks odd.

Expected behaviour

Icons and text in a button should be at the same alignment.

Steps To Reproduce

Screenshot 2021-05-25 at 6 24 44 PM

You can see the icon is slightly above the text (I can also confirm that the text is correctly centred but the icon is not). It is true for all icons and checked with different texts and they all look weird.

@vopani vopani added the bug Bug in code label May 25, 2021
@mturoci mturoci added the ui Related to UI label May 26, 2021
@mturoci mturoci self-assigned this May 26, 2021
@mturoci
Copy link
Collaborator

mturoci commented May 27, 2021

It is true for all icons

Not sure about this. Take Search icon for example:
image

The reason some icons are centered and some not depends on how they are clipped. The icon from your example NavigationBack has extra bottom space, which causes the visual inconsistence (It appears slightly up).

image

On the other hand, Search is clipped perfectly:
image
Same goes for Add:
image

I guess there is not much we can do about this as these is a bug of the Fluent Icon font.

@vopani
Copy link
Contributor Author

vopani commented May 28, 2021

:-(

@mturoci
Copy link
Collaborator

mturoci commented Jul 21, 2022

Seems like this is now possible to fix due to switching to SVG icons instead of font icons.

@vopani
Copy link
Contributor Author

vopani commented Jul 22, 2022

@mturoci Can you share an example here for reference purposes? Thanks!

@mturoci
Copy link
Collaborator

mturoci commented Jul 22, 2022

Sure.

Before
image
After
image

#837 (comment) still stands as some icons are not cropped properly, making it impossible to achieve centering perfection.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug in code ui Related to UI
Projects
None yet
Development

No branches or pull requests

2 participants