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

fix(settings): set viewBox on SVG icons #42822

Merged
merged 2 commits into from
Jan 22, 2024
Merged

Conversation

ShGKme
Copy link
Contributor

@ShGKme ShGKme commented Jan 16, 2024

Summary

Added missing viewBox to SVG icons in settings. Without viewBox they don't scale.

For example, on the Users page when there are no users.

No changes in other places.

🏚️ Before 🏑 After
image image

Checklist

@ShGKme ShGKme added this to the Nextcloud 29 milestone Jan 16, 2024
@ShGKme ShGKme self-assigned this Jan 16, 2024
@ShGKme
Copy link
Contributor Author

ShGKme commented Jan 16, 2024

/backport 6179aed stable28

@giantmolecules1

This comment was marked as off-topic.

Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense

@ShGKme
Copy link
Contributor Author

ShGKme commented Jan 16, 2024

Drone is not related, I cannot restart it...

Without viewBox they don't scale

Signed-off-by: Grigorii K. Shartsev <[email protected]>
Signed-off-by: Grigorii K. Shartsev <[email protected]>

chore(assets): Recompile assets

Signed-off-by: Grigorii K. Shartsev <[email protected]>
@ShGKme ShGKme merged commit d133fce into master Jan 22, 2024
44 checks passed
@ShGKme ShGKme deleted the fix/settings-svg-viewBox branch January 22, 2024 23:14
@ShGKme
Copy link
Contributor Author

ShGKme commented Jan 23, 2024

/backport f5ec894 stable28

@ShGKme
Copy link
Contributor Author

ShGKme commented Jan 23, 2024

/backport f5ec894 to stable28

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

Successfully merging this pull request may close these issues.

5 participants