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

Nav > mobile > blured svg default icons #44706

Closed
Marc-pi opened this issue Oct 5, 2022 · 3 comments
Closed

Nav > mobile > blured svg default icons #44706

Marc-pi opened this issue Oct 5, 2022 · 3 comments

Comments

@Marc-pi
Copy link

Marc-pi commented Oct 5, 2022

Description

Actually, we cannot set from the block settings the navigation icon size (see #37930). The default icon size is 24x24px (see source, the svg is inlined).

If we enlarge it with css to 50x50px, it is blured.

Seems to be a png file converted to svg. the same for the other icon (burger with 2 or 3 lines = same)
Maybe the design team can generate a cristal clear code to replace the 2 actual default icons
for WP6.1 final !

Step-by-step reproduction instructions

Just set your nav and active the burger icon for mobile.
change the default css, for instance to 50px
image

Screenshots, screen recording, code snippet

image

Environment info

WP6.0.2 with Gutenberg plugin 14.2.0

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

@jasmussen
Copy link
Contributor

Hey, thanks for the issue. Can you try and make the icon 48x48? That would be a 2x integer scale from the 24x24px base icon, and should remove any blur you see.

@Marc-pi
Copy link
Author

Marc-pi commented Oct 5, 2022

hum, first time i notice this with svg (using interger multiple is needed) , works better, thanks for your help
i close

@Marc-pi Marc-pi closed this as completed Oct 5, 2022
@jasmussen
Copy link
Contributor

Glad that worked!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants