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(select): visual fixes over long texts #355

Merged

Conversation

muratcorlu
Copy link
Contributor

This fixes several visual issues about checkbox and select components that is reported in #353 and #352

  • Checkbox checkmark background color
  • Checkbox checkmark deformation with limited space
  • Checkbox label with long text and multiple lines
  • Select component long placeholder and labels

I tried to add extra stories to our storybook to cover those situations in our visual tests.

Fixes #353
Fixes #352

@muratcorlu muratcorlu linked an issue Jan 5, 2023 that may be closed by this pull request
@sonarcloud
Copy link

sonarcloud bot commented Jan 5, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Member

@leventozen leventozen left a comment

Choose a reason for hiding this comment

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

Great job!
I highly agreed with adding break-all is a proper way of wrapping a long text's for checkboxes (or for all inputs). Since those are represents the choices of the users.

@muratcorlu muratcorlu merged commit 84ce4cd into next Jan 6, 2023
@muratcorlu muratcorlu deleted the 353-no-background-color-checkmark-of-checkbox-component branch January 6, 2023 13:34
@github-actions
Copy link

github-actions bot commented Jan 6, 2023

🎉 This PR is included in version 2.0.0-beta.57 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link

github-actions bot commented Apr 6, 2023

🎉 This PR is included in version 2.0.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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No background color checkmark of checkbox component Baklava Multiple Select Label, Checkbox ui problem
2 participants