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

[VO-1194] feat(SquareAppIcon): Hide shortcut badges for a more app-like feel #2717

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

cballevre
Copy link
Member

By default, the badges are displayed for the shortcut variant, but in some cases you want to be able to hide them so that they look more like applications

Demo : https://cballevre.github.io/cozy-ui/react/#!/Extra/SquareAppIcon

Copy link

bundlemon bot commented Nov 15, 2024

BundleMon

Unchanged files (3)
Status Path Size Limits
dist/cozy-ui.min.css
20.23KB +10%
dist/cozy-ui.utils.min.css
10.91KB +10%
transpiled/react/stylesheet.css
4.83KB +10%

No change in files bundle size

Groups updated (1)
Status Path Size Limits
transpiled/react/**
645.32KB (+56B +0.01%) -

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

@cballevre cballevre force-pushed the feat/hide-shortcut-badge branch 2 times, most recently from 0694706 to 14b8d1a Compare November 15, 2024 14:19
By default, the badges are displayed for the shortcut variant, but in some cases you want to be able to hide them so that they look more like applications
Copy link
Contributor

@zatteo zatteo left a comment

Choose a reason for hiding this comment

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

👍 👍

@cballevre cballevre merged commit 49d8438 into master Nov 18, 2024
12 checks passed
@cballevre cballevre deleted the feat/hide-shortcut-badge branch November 18, 2024 08:22
@cozy-bot
Copy link

🎉 This PR is included in version 113.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants