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.

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

feat: added glareBorderRadius prop to stop glare overlapping rounded children #28

Merged
merged 1 commit into from
Apr 3, 2021

Conversation

LucidNinja
Copy link
Contributor

Description
Adds a new prop (glareBorderRadius) to handle an issue where glare corners were visible when the background color of the app and the glare color were different, if the child component had rounded edges.

fix #27

Checklist

  • Commit messages should follow conventional commits convention as much as possible. After staging your changes please run npm run commit
  • Lint, prettier and all tests passing - npm run validate
  • Extended the Storybook demo page / README / documentation, if necessary

…children

The border radius of the glare can now be the same as the child component. This previously presented
as an issue when the glare color was different to the background color of the app.

fix mkosir#27
@mkosir mkosir self-requested a review April 3, 2021 19:15
@mkosir mkosir added bug Something isn't working enhancement New feature or request labels Apr 3, 2021
Copy link
Owner

@mkosir mkosir left a comment

Choose a reason for hiding this comment

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

PR looks great 🚀

@mkosir mkosir merged commit 560736d into mkosir:master Apr 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Doesn't support rounded edges (or other glare props)
2 participants