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

Extra space before spinner SVG parameter values causing error to be logged in Safari #16089

Closed
thexeos opened this issue Jul 19, 2023 · 1 comment · Fixed by #16090
Closed

Comments

@thexeos
Copy link
Contributor

thexeos commented Jul 19, 2023

What happened?

In particular example, QSpinnerCube has spaces before some width values, such as these:

They trigger an error to be logged in some Safari versions:

image

This also occurs with QSpinnerComment, QSpinnerFacebook, and QSpinnerHourglass.

What did you expect to happen?

No error

Reproduction URL

NA

How to reproduce?

NA

Flavour

Quasar CLI with Vite (@quasar/cli | @quasar/app-vite)

Areas

Components (quasar)

Platforms/Browsers

Safari

Quasar info output

No response

Relevant log output

No response

Additional context

No response

@github-actions
Copy link

Hi @thexeos! 👋

It looks like you provided an invalid or unsupported reproduction URL.
Do not use any service other than Codepen, jsFiddle, StackBlitz, Codesandbox, and GitHub.
Make sure the URL you provided is correct and reachable. You can test it by visiting it in a private tab, another device, etc.
Please edit your original post above and provide a valid reproduction URL as explained.

Without a proper reproduction, your issue will have to get closed.

Thank you for your collaboration. 👏

@github-actions github-actions bot added bug/0-needs-info Need more info to reproduce flavour/quasar-cli-vite labels Jul 19, 2023
@thexeos thexeos mentioned this issue Jul 19, 2023
15 tasks
rstoenescu pushed a commit that referenced this issue Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant