Skip to content

fix: mask-size for brand icons#197

Merged
lindsayzhng merged 1 commit intomainfrom
dmytro/icon-mask-size-style
Feb 12, 2026
Merged

fix: mask-size for brand icons#197
lindsayzhng merged 1 commit intomainfrom
dmytro/icon-mask-size-style

Conversation

@pqoqubbw
Copy link
Contributor

@pqoqubbw pqoqubbw commented Feb 10, 2026

Summary

Test Plan


Note

Low Risk
Small, localized styling change limited to icon rendering; risk is mainly visual regressions in specific icon variants.

Overview
Fixes Icon SVG masking for Lucide brand icons by conditionally applying maskSize: "100%" only when iconLibrary is lucide and the icon is not a Font Awesome brand (via isBrandsIcon).

This prevents brand icons from being scaled incorrectly while keeping the previous sizing behavior for other Lucide icons.

Written by Cursor Bugbot for commit 0e2e993. This will update automatically on new commits. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@lindsayzhng lindsayzhng force-pushed the dmytro/icon-mask-size-style branch from 2aeaed3 to 0e2e993 Compare February 12, 2026 21:19
@lindsayzhng lindsayzhng merged commit 61b2bd8 into main Feb 12, 2026
3 checks passed
@lindsayzhng lindsayzhng deleted the dmytro/icon-mask-size-style branch February 12, 2026 21:22
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.

2 participants