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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: add fill-rule to expressive icons lacking currentColor #250

Merged
merged 1 commit into from
Nov 20, 2023
Merged

fix: add fill-rule to expressive icons lacking currentColor #250

merged 1 commit into from
Nov 20, 2023

Conversation

skjalgepalg
Copy link
Collaborator

Found three more icons missing fill="currentColor"

@skjalgepalg skjalgepalg added the semver:patch Involves backwards compatible bug fixes label Nov 20, 2023
@skjalgepalg skjalgepalg requested a review from phajsi November 20, 2023 12:17
@skjalgepalg skjalgepalg self-assigned this Nov 20, 2023
phajsi
phajsi approved these changes Nov 20, 2023
View reviewed changes
Copy link
Contributor

@phajsi phajsi left a comment

Choose a reason for hiding this comment

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

Nice 💯

@skjalgepalg skjalgepalg merged commit 3e2e80c into master Nov 20, 2023
1 check passed
@skjalgepalg skjalgepalg deleted the missing-fill branch November 20, 2023 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@phajsi phajsi phajsi approved these changes

Assignees

@skjalgepalg skjalgepalg

Labels
semver:patch Involves backwards compatible bug fixes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@skjalgepalg @phajsi