-
Notifications
You must be signed in to change notification settings - Fork 14
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
feat(sbb-form-field): support the textarea inside the <sbb-form-field>
#2506
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is LGTM except for the "not focus" rules which feel a bit of a workaround 😞
Are they unavoidable? Why is it just the textarea that has problems?
6226749
to
dacef49
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #2506 +/- ##
=======================================
Coverage ? 93.20%
=======================================
Files ? 301
Lines ? 24930
Branches ? 2031
=======================================
Hits ? 23237
Misses ? 1664
Partials ? 29 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice solution. some topics open
1cd0b5f
to
8828606
Compare
1f29663
to
5c2e50f
Compare
This reverts commit ac27214.
28ae767
to
198b345
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Nice work!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM :)
Closes #2497
BREAKING CHANGE: The css var
--sbb-form-field-height
has been renamed to--sbb-form-field-min-height
.