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

feat(v5): Add floating labels #5567

Merged
merged 8 commits into from
May 8, 2021
Merged

feat(v5): Add floating labels #5567

merged 8 commits into from
May 8, 2021

Conversation

kyletsang
Copy link
Member

@kyletsang kyletsang commented Dec 16, 2020

https://getbootstrap.com/docs/5.0/forms/floating-labels/

These are initial thoughts on the floating label API.

There's 2 approaches to creating floating labels:

  1. Using a FloatingLabel and wrapping an input element. This approach takes care of rendering the label and passing the control id down to the input/label.
  2. Using FormFloating and wrapping an input element and a label. This is to allow customization in case the above doesn't work out.

@kyletsang kyletsang added this to In Progress in v5 support via automation Dec 16, 2020
@kyletsang kyletsang marked this pull request as draft December 16, 2020 23:19
@kyletsang kyletsang marked this pull request as ready for review December 24, 2020 18:46
@kyletsang kyletsang requested a review from jquense March 31, 2021 07:02
Copy link
Member

@jquense jquense left a comment

Choose a reason for hiding this comment

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

this looks great

@jquense
Copy link
Member

jquense commented May 8, 2021

maybe worth adding a few conveniences to some of the form controls to have typing but I think this is a good base API at least

@kyletsang kyletsang merged commit 6802774 into bs5-dev May 8, 2021
v5 support automation moved this from In Progress to Done May 8, 2021
@kyletsang kyletsang deleted the v5-floating-label branch May 8, 2021 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
v5 support
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants