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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(Bootstrap): remove deprecated classes #1815

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

anrolmar
Copy link
Contributor

Describe the PR

This PR remove all the deprecated classes mentioned on #1779 (comment)

PR checklist

What kind of change does this PR introduce? (check at least one)

  • Bugfix 馃悰 - fix(...)
  • Feature - feat(...)
  • ARIA accessibility - fix(...)
  • Documentation update - docs(...)
  • Other (please describe)

The PR fulfills these requirements:

  • Pull request title and all commits follow the Conventional Commits convention or has an override in this pull request body This is very important, as the CHANGELOG is generated from these messages, and determines the next version type. Pull requests that do not follow conventional commits or do not have an override will be denied

Copy link

stackblitz bot commented Mar 15, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

remove .input-group-append and .input-group-prepend classes
update the documentation for BInputGroup
Copy link
Contributor

This PR is stale because it has been open for 45 days with no activity. It will not be auto-closed

@github-actions github-actions bot added the stale There has been no additional replies or questions and the thread is assumed closed label May 30, 2024
@xvaara
Copy link
Contributor

xvaara commented May 30, 2024

@anrolmar @dwgray @VividLemon is this pr still in need of changes?

@@ -67,8 +67,6 @@ Easily extend form controls by adding text, buttons, or button groups on either

You can attach addons using either props, named slots and/or sub-components.

**Note** Bootstrap 5 dropped `.input-group-append` and `.input-group-prepend`. You can now just add buttons and `.input-group-text` as direct children of the input groups. The sub-components are added for compatibility reasons.
Copy link
Member

Choose a reason for hiding this comment

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

@xvaara It seems like if this has been dropped, that we could remove the sub-components -- I noticed this the other day, that the two components were exactly the same

Copy link
Member

Choose a reason for hiding this comment

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

I removed these two components in my feature branch

@github-actions github-actions bot removed the stale There has been no additional replies or questions and the thread is assumed closed label May 31, 2024
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.

None yet

4 participants