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

fix(ProgressIndicator): Avoid downstream runtime warning from Next.js #3772

Merged

Conversation

m7kvqbe1
Copy link
Collaborator

@m7kvqbe1 m7kvqbe1 commented Apr 30, 2024

Related issue

Closes #3714

Overview

Apply via css property in keyframe definition to avoid inline transform-origin upon on transpilation.

Reason

Next.js is wrongly throwing a runtime warning about the transform-origin SVG attribute.

This is a valid HTML SVG attribute.

Work carried out

  • Upgrade @svgr/cli to latest
  • Avoid inline transform-origin usage
  • Fix loader.svg code formatting

Screenshot

Screenshot 2024-04-30 at 11 57 14

Developer notes

vercel/next.js#53342 (comment)

https://github.com/facebook/react/pull/26130/files#diff-67b481b96d526280780f010dfd192761f2d1f6dc812e3015020a7986501d2ebaR524

@m7kvqbe1 m7kvqbe1 added Type: Bug Inconsistencies or issues which have caused a problem for users or implementors Package: react-component-library Package/code type Package: icon-library Package/code type labels Apr 30, 2024
@m7kvqbe1 m7kvqbe1 requested a review from will-lamb April 30, 2024 10:58
@m7kvqbe1 m7kvqbe1 self-assigned this Apr 30, 2024
@m7kvqbe1 m7kvqbe1 requested a review from markhigham as a code owner April 30, 2024 10:58
Copy link

netlify bot commented Apr 30, 2024

Deploy Preview for storybook-navy-digital-mod-uk ready!

Name Link
🔨 Latest commit b287d42
🔍 Latest deploy log https://app.netlify.com/sites/storybook-navy-digital-mod-uk/deploys/66321f46e1617f00086997a4
😎 Deploy Preview https://deploy-preview-3772--storybook-navy-digital-mod-uk.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@m7kvqbe1 m7kvqbe1 force-pushed the fix/transform-origin-svg-attribute-next-runtime-error branch from a9afe74 to 3505d1d Compare April 30, 2024 11:00
Copy link
Collaborator

@markhigham markhigham left a comment

Choose a reason for hiding this comment

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

Some merge conflicts. Changes LGTM

@m7kvqbe1 m7kvqbe1 force-pushed the fix/transform-origin-svg-attribute-next-runtime-error branch from 3505d1d to e7c32a2 Compare May 1, 2024 10:50
Copy link

sonarcloud bot commented May 1, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@m7kvqbe1 m7kvqbe1 merged commit e440509 into master May 1, 2024
21 of 23 checks passed
@m7kvqbe1 m7kvqbe1 deleted the fix/transform-origin-svg-attribute-next-runtime-error branch May 1, 2024 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Package: icon-library Package/code type Package: react-component-library Package/code type Type: Bug Inconsistencies or issues which have caused a problem for users or implementors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect transform-origin prop in IconLoader
2 participants