Skip to content

Conversation

@Jialecl
Copy link
Collaborator

@Jialecl Jialecl commented May 3, 2024

Checklist
(Check off all the items before submitting)

  • Build process is done without errors. All tests pass in the /lib directory.
  • Self-reviewed the code before submitting.
  • Meets accessibility standards.
  • Added/updated documentation to /website as needed.
  • Added/updated tests as needed.

Purpose
This PR fixes some strange behavior in the size of some of our input components.
Instead of components with 100% width being pushed by other components with a fixed width it was the other way around.
The password and number were not behaving correctly inside a flex component when the size was set to fillParent.

Closes #1970

@Mil4n0r Mil4n0r self-requested a review May 3, 2024 11:57
@Mil4n0r Mil4n0r self-assigned this May 3, 2024
Copy link
Collaborator

@Mil4n0r Mil4n0r left a comment

Choose a reason for hiding this comment

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

I have seen that there are multiple commas added, this is due to the trailingComma parameter not being set for prettier. It has no impact and will be included in the Code Quality PR though.

@Mil4n0r Mil4n0r merged commit 50acff4 into master May 6, 2024
@Mil4n0r Mil4n0r deleted the jialecl/inputs-size-fix branch May 6, 2024 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Select and textInput size is not correct in a certain scenario

3 participants