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

uui-color-swatches: color swatches changes form when showLabel is true #427

Merged
merged 4 commits into from
Mar 21, 2023

Conversation

loivsen
Copy link
Contributor

@loivsen loivsen commented Feb 22, 2023

Color swatches changes form when showLabel is set to true.
Added logic for overflow on swatches labels
Color swatches are now an actual button
Removed extra padding for swatches
Added padding for color picker's swatch area

Copy link
Member

@nielslyngsoe nielslyngsoe left a comment

Choose a reason for hiding this comment

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

Hi Lone, I have added a few comments here and there. I hope they make sense, Thanks

@nielslyngsoe nielslyngsoe enabled auto-merge (squash) March 9, 2023 10:29
@nielslyngsoe
Copy link
Member

@loivsen is seems there is a test failing. Will you have a look at it?.. I have approved it, so once it goes through it should be merged. :-)

@iOvergaard iOvergaard added the feature Something we are looking to build label Mar 21, 2023
@iOvergaard iOvergaard changed the title Color swatches changes form when showLabel is true uui-color-swatches: color swatches changes form when showLabel is true Mar 21, 2023
@iOvergaard iOvergaard merged commit b42a474 into v1/contrib Mar 21, 2023
@iOvergaard iOvergaard deleted the bugfix/color-swatches-label branch March 21, 2023 12:44
@iOvergaard iOvergaard linked an issue Mar 22, 2023 that may be closed by this pull request
iOvergaard added a commit that referenced this pull request Apr 20, 2023
…is true (#427)

* Color swatches changes form when showLabel is true

---------

Co-authored-by: Niels Lyngsø <[email protected]>
Co-authored-by: Jacob Overgaard <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Something we are looking to build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Swatches with labels are not shown in the best way
3 participants