Skip to content

fix: bootstrap icon sizes#4121

Merged
leafty merged 2 commits into
mainfrom
leafty/bootstrap-icon-size
Apr 9, 2026
Merged

fix: bootstrap icon sizes#4121
leafty merged 2 commits into
mainfrom
leafty/bootstrap-icon-size

Conversation

@leafty
Copy link
Copy Markdown
Member

@leafty leafty commented Apr 9, 2026

Fixes the icon size for Bootstrap icons.

Icon size (for bootstrap icons) should now be controlled with the fontSize prop. The .bi class is added by the react-bootstrap-icons package now so the icons are forced to be the same size as the font. (So to change the icon size, just change the font size).

/deploy renku=release-2.16.0

@leafty leafty marked this pull request as ready for review April 9, 2026 09:14
@leafty leafty requested a review from a team as a code owner April 9, 2026 09:14
@RenkuBot
Copy link
Copy Markdown
Contributor

RenkuBot commented Apr 9, 2026

You can access the deployment of this PR at https://renku-ci-ui-4121.dev.renku.ch

Copy link
Copy Markdown
Member

@lorenzo-cavazzi lorenzo-cavazzi left a comment

Choose a reason for hiding this comment

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

Nice fix 🎉

P.S. We probably want to handle sizes through classes like fs-x instead of fixed numbers, but the PR restores the previous state which is already great 👍

@leafty leafty enabled auto-merge (squash) April 9, 2026 09:43
@leafty leafty merged commit 5ec623e into main Apr 9, 2026
14 of 23 checks passed
@leafty leafty deleted the leafty/bootstrap-icon-size branch April 9, 2026 09:46
@RenkuBot
Copy link
Copy Markdown
Contributor

RenkuBot commented Apr 9, 2026

Tearing down the temporary RenkuLab deplyoment for this PR.

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