Skip to content

Conversation

@jonny5
Copy link
Collaborator

@jonny5 jonny5 commented Sep 16, 2025

Description

  • _form_floating_labels.scss is based on Bootstrap 4. In Bootstrap 4 the variable input-border-width was defined as a set amount of pixels, it is now var(--bs-border-width) which needs calc to multiply by 2.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

$input-border-width is set to `var(--bs-border-width)` which cannot be directly multiplied
@cielf
Copy link
Collaborator

cielf commented Sep 16, 2025

This seems to have addressed my issue -- thanks!

@cielf cielf requested a review from dorner September 16, 2025 18:06
@awwaiid
Copy link
Collaborator

awwaiid commented Sep 21, 2025

Any guess how I can reproduce this on my system? I haven't run into it, so hard to verify that it is fixed :)

Copy link
Collaborator

@awwaiid awwaiid left a comment

Choose a reason for hiding this comment

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

Well I confirm that it doesn't make it worse and the documentation agrees and this is the only instance.

@awwaiid awwaiid merged commit b5b634d into rubyforgood:main Nov 30, 2025
11 of 12 checks passed
@github-actions
Copy link
Contributor

@jonny5: Your PR Fix error when variable is computed is part of today's Human Essentials production release: 2025.11.30.
Thank you very much for your contribution!

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.

3 participants