Problem/Motivation

Drupal core provides a form-text class to some fields (i. e. text fields). form-text class also has a special meaning in Bootstrap. So this collides in some points and Bootstrap5 theme is setting $form-text-margin-top variable to fix this issue I suppose.
But this does also affect the "regular" use of the Bootstrap form-text class, and you may not be able to easily override the variable yourself.

Steps to reproduce

Proposed resolution

I would propose to not set the Bootstrap variable, but instead fix the issue in the Drupal styles, by setting a zero margin to input.form-text selector. This fixes the issue with the form-text class from Drupal, but still let us make use of the Bootstrap form-text class if we want to and override the Bootstrap variable in case.

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork bootstrap5-3483421

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

stefan.korn created an issue. See original summary.

stefan.korn’s picture

Status: Active » Needs review

stefan.korn’s picture

reroll with latest head

jannakha’s picture

Status: Needs review » Reviewed & tested by the community

thank you for your contribution.
tested and ready to be merged in.

stefan.korn’s picture

reroll with latest head

vladimiraus’s picture

Status: Reviewed & tested by the community » Fixed

Thank you! Committed.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • vladimiraus committed 4319c791 on 4.0.x
    feat: #3483421 Less invasive fix for form-text class problem
    
    By: stefan...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.