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

[Bug Report][3.7.7] VImg: Bug in order of specification of crossOrigin property in mac Safari #20914

Closed
kzfk opened this issue Jan 24, 2025 · 0 comments · Fixed by #20915
Closed

[Bug Report][3.7.7] VImg: Bug in order of specification of crossOrigin property in mac Safari #20914

kzfk opened this issue Jan 24, 2025 · 0 comments · Fixed by #20915
Labels
C: VImg platform specific The issue only occurs on a specific platform T: bug Functionality that does not work as intended/expected
Milestone
v3.7.x

Comments

@kzfk
Copy link
Contributor

kzfk commented Jan 24, 2025

Environment

Vuetify Version: 3.7.7
Vue Version: 3.5.13
Browsers: Safari
OS: iOS, Mac OSX

Steps to reproduce

Using VImg components in the above environment.
Use Safari's Web Inspector to see how many times the image is being called on the network.

Expected Behavior

Image should load once on Safari, with Origin header present if crossorigin attribute is set.

Actual Behavior

Image is being loaded twice on Safari, first time without Origin header, second time with Origin header.

Reproduction Link

https://play.vuetifyjs.com/#...

Other comments

Needs to be corrected based on information in vuejs/core#4680.
kzfk added a commit to kzfk52/vuetify that referenced this issue Jan 24, 2025
@kzfk
fix(VImg): Bug in order of specification of crossOrigin property in m…
588361e 
…ac Safari

fixed vuetifyjs#20914
@kzfk kzfk mentioned this issue Jan 24, 2025
Merged
@KaelWD KaelWD added T: bug Functionality that does not work as intended/expected platform specific The issue only occurs on a specific platform C: VImg labels Jan 24, 2025
@KaelWD KaelWD closed this as completed in 910935e Jan 24, 2025
@KaelWD KaelWD added this to the v3.7.x milestone Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C: VImg platform specific The issue only occurs on a specific platform T: bug Functionality that does not work as intended/expected
Projects
None yet
Milestone
v3.7.x
Development

Successfully merging a pull request may close this issue.

fix(VImg): set crossOrigin attribute before src kzfk52/vuetify
2 participants
@kzfk @KaelWD