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: Fix InputGroup inner end width calculations #2984

Merged

Conversation

NicholasBoll
Copy link
Member

Summary

Fixes InputGroup.InnerEnd width and positioning calculations. Without this change, InnerEnd elements are rendered in the wrong location.

Release Category

Components


Checklist

  • Label ready for review has been added to PR

Screenshots or GIFs (if applicable)

Variable widths before this change:
image

Variable widths after this change:
image

@NicholasBoll NicholasBoll added the ready for review Code is ready for review label Oct 13, 2024
@github-actions github-actions bot requested a review from alanbsmith October 13, 2024 04:01
Copy link

cypress bot commented Oct 13, 2024

Workday/canvas-kit    Run #7895

Run Properties:  status check passed Passed #7895  •  git commit e81af9cc73 ℹ️: Merge da1caeaae7f873c1a071a42224ff1f97199dae40 into 23aaa97b9bc903bc333d2baa1654...
Project Workday/canvas-kit
Run status status check passed Passed #7895
Run duration 03m 59s
Commit git commit e81af9cc73 ℹ️: Merge da1caeaae7f873c1a071a42224ff1f97199dae40 into 23aaa97b9bc903bc333d2baa1654...
Committer Nicholas Boll
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 32
Tests that did not run due to a developer annotating a test with .skip  Pending 24
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 1082
UI Coverage  21.68%
  Untested elements 1642  
  Tested elements 452  
Accessibility  99.18%
  Failed rules  5 critical   5 serious   0 moderate   2 minor
  Failed elements 182  

@NicholasBoll NicholasBoll added automerge and removed ready for review Code is ready for review labels Oct 14, 2024
@alanbsmith alanbsmith enabled auto-merge (squash) October 14, 2024 02:03
@alanbsmith alanbsmith merged commit b361815 into Workday:prerelease/major Oct 14, 2024
18 of 20 checks passed
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.

4 participants