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(NcCounterBubble): adjust min-width to make it a right circle #5905

Merged
merged 1 commit into from
Aug 5, 2024

Conversation

ShGKme
Copy link
Contributor

@ShGKme ShGKme commented Aug 4, 2024

☑️ Resolves

  • Fixes: with small numbers like 1 NcCounterBubble is narrower than a circle
  • Makes min-width equal to height

Note: it is a but not centered vertically, because actually digits in fonts are smaller than line height. It will be fixed in a separate PR.

🖼️ Screenshots

🏚️ Before 🏡 After
image image
image image
image image

🏁 Checklist

  • ⛑️ Tests are included or are not applicable
  • 📘 Component documentation has been extended, updated or is not applicable
  • 3️⃣ Backport to next requested with a Vue 3 upgrade

@ShGKme ShGKme added bug Something isn't working 3. to review Waiting for reviews labels Aug 4, 2024
@ShGKme ShGKme self-assigned this Aug 4, 2024
@ShGKme
Copy link
Contributor Author

ShGKme commented Aug 4, 2024

/backport to next

@ShGKme ShGKme merged commit a474ec2 into master Aug 5, 2024
19 checks passed
@ShGKme ShGKme deleted the fix/nc-counter-button--min-width branch August 5, 2024 08:29
@Antreesy Antreesy mentioned this pull request Aug 5, 2024
@ShGKme ShGKme changed the title fix(NcCounterButton): adjust min-width to make it a right circle fix(NcCounterBubble): adjust min-width to make it a right circle Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants