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

Going back on browser shows past variation #9834

Closed
exezbcz opened this issue Mar 21, 2024 · 14 comments · Fixed by #9867
Closed

Going back on browser shows past variation #9834

exezbcz opened this issue Mar 21, 2024 · 14 comments · Fixed by #9867
Assignees
Labels
A-drop-page issues related to drop page p2 core functionality, or is affecting 60% of app UX first Improvement for UX

Comments

@exezbcz
Copy link
Member

exezbcz commented Mar 21, 2024

Screen.Recording.2024-03-21.at.13.31.16.mov

this was not happening before, you should get to the page that you were before

how to reproduce: generate few variations and click back

@exezbcz exezbcz added UX first Improvement for UX p2 core functionality, or is affecting 60% of app A-drop-page issues related to drop page labels Mar 21, 2024
@Jarsen136 Jarsen136 self-assigned this Mar 21, 2024
@kodabot
Copy link
Collaborator

kodabot commented Mar 21, 2024

ASSIGNED - @Jarsen136 🔒 LOCKED -> Saturday, March 23rd 2024, 24:37:39 UTC -> 36 hours

@Jarsen136
Copy link
Contributor

I do not have an ideal solution as it's more of browser behaviour.

@Jarsen136 Jarsen136 removed their assignment Mar 22, 2024
@kodabot
Copy link
Collaborator

kodabot commented Mar 22, 2024

ASSIGNMENT OVER - @Jarsen136 has been unassigned.

@exezbcz
Copy link
Member Author

exezbcz commented Mar 23, 2024

@Jarsen136 and did it happen before? Because i think previously the back button was working as expected

@daiagi
Copy link
Contributor

daiagi commented Mar 23, 2024

this is strange
it is definitely this line

generativeImageUrl.value = previewItem.image

but since when does setting a value to a ref pushes into history?

also, no affect on url or on history API

https://developer.mozilla.org/en-US/docs/Web/API/History_API

@Jarsen136
Copy link
Contributor

@Jarsen136 and did it happen before? Because i think previously the back button was working as expected

Yes, it's the same as before. It's related to the iframe would effect the browser history.

Here is the preview env from two months ago. https://deploy-preview-9343--koda-beta.netlify.app/ahp/drops/convergente

@daiagi
Copy link
Contributor

daiagi commented Mar 23, 2024

@Jarsen136

am i blind? i don't find an Iframe in the page

image

@Jarsen136
Copy link
Contributor

@Jarsen136

am i blind? i don't find an Iframe in the page

search the iframe element in HTML

image

@daiagi
Copy link
Contributor

daiagi commented Mar 23, 2024

weird. it's an image on localhost and iframe on production

@daiagi
Copy link
Contributor

daiagi commented Mar 23, 2024

where is the sandbox attribute determined?

didn't see anything about it in MediaItem.vue

maybe can use this?

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/iframe#allow-top-navigation

@Jarsen136
Copy link
Contributor

weird. it's an image on localhost and iframe on production

it should be the same as localhost.
image

where is the sandbox attribute determined?

didn't see anything about it in MediaItem.vue

IFrameMedia.vue

maybe can use this?

Feel free to delve deeper and give it a shot

@daiagi
Copy link
Contributor

daiagi commented Mar 23, 2024

doesn't work
i removed all permissions

image

didn't do a thing

@daiagi
Copy link
Contributor

daiagi commented Mar 23, 2024

i'll try something else

@daiagi daiagi self-assigned this Mar 23, 2024
@kodabot
Copy link
Collaborator

kodabot commented Mar 23, 2024

ASSIGNED - @daiagi 🔒 LOCKED -> Sunday, March 24th 2024, 22:02:19 UTC -> 36 hours

@daiagi daiagi mentioned this issue Mar 23, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-drop-page issues related to drop page p2 core functionality, or is affecting 60% of app UX first Improvement for UX
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants