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

Fixing strokeWidth typo for Warning Icon #5030

Open
wants to merge 1 commit into
base: canary-patch
Choose a base branch
from

Conversation

zivc
Copy link

@zivc zivc commented May 22, 2022

The strokeWidth attribute for this Path component was incorrectly defined as stroke-width.

Error in console;
Warning: Invalid DOM property `stroke-width`. Did you mean `strokeWidth`? at path at g at ChildPath (http://localhost:19006/static/js/bundle.js:65185:23) at g at svg at http://localhost:19006/static/js/bundle.js:88165:22 at SVGIcon (http://localhost:19006/static/js/bundle.js:65150:23) at createdIcon at div at http://localhost:19006/static/js/bundle.js:126663:25 at http://localhost:19006/static/js/bundle.js:88165:22 at Box (http://localhost:19006/static/js/bundle.js:63369:23) at AlertIcon (http://localhost:19006/static/js/bundle.js:53800:23)

The `strokeWidth` attribute for this `Path` component was incorrectly defined as `stroke-width`.
@auto-assign auto-assign bot requested a review from surajahmed May 22, 2022 00:59
@vercel
Copy link

vercel bot commented May 22, 2022

@zivc is attempting to deploy a commit to the Geekyants Team Team on Vercel.

A member of the Team first needs to authorize it.

@zivc
Copy link
Author

zivc commented May 28, 2022

@surajahmed any update?

@exneval
Copy link
Contributor

exneval commented Jul 12, 2022

duplicate of #4932

@zivc
Copy link
Author

zivc commented Jul 12, 2022

duplicate of #4932

Good luck getting anyones attention

@Viraj-10 Viraj-10 changed the base branch from master to canary-patch August 4, 2022 10:07
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