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

fixed an issue caused by autowidth #557

Merged
merged 2 commits into from
Aug 23, 2023

Conversation

loivsen
Copy link
Contributor

@loivsen loivsen commented Aug 21, 2023

Bugfix for uui-input

Types of changes

In the scenario of uui-input being set to 100% and having an append attached to it, there is a visual bug where the appended element doesn't go to the end. The new auto-width feature causes this bug.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Chore (minor updates related to the tooling or maintenance of the repository, does not impact compiled assets)

Motivation and context

How to test?

Screenshots (if appropriate)

Before fix (current)
before
After fix (pull request)
after

@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://delightful-beach-055ecb503-557.westeurope.azurestaticapps.net

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.

Looks good

@loivsen loivsen enabled auto-merge August 23, 2023 12:37
@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://delightful-beach-055ecb503-557.westeurope.azurestaticapps.net

@loivsen loivsen merged commit 04ef2e2 into v1/contrib Aug 23, 2023
7 of 8 checks passed
@loivsen loivsen deleted the bugfix/uui-input-maxwidth-and-append branch August 23, 2023 12:48
@iOvergaard iOvergaard added the bug Something isn't working label Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants