-
Notifications
You must be signed in to change notification settings - Fork 400
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
Embedded geostory URL issue with scroll position #9047
Comments
Thank you @ElenaGallo. @dsuren1 can you please investigate/estimate this? |
Investigation The scroll information is removed from the shared url, only when
We can add a specific config in
@tdipisa Kindly let me know your thoughts. |
@dsuren1 thank you.
I'm sorry but it is not clear to me. What do you mean precisely? The expected behavior in this case would not be to 'remove' the section path in case of embedded URL. The expected behavior would be to have it working by scrolling to the desired section also for embedded gestories. |
As per the issue description, the expected result is
|
@dsuren1 If you could spend 1h more to identify the reason of the regression providing also an estimate for the fix that would be good. |
From the gif, you can see that section id is not added when user is on the first section but added when scrolled to second. This is because the section id and column id is parsed from the url of the page here instead of the fetching the value from There are three issues,
This behavior exist on previous versions as well. This scenario can be easy to miss and probably overlooked and not happening just recently. |
Description
The embedded geostory is not visible if the Advanced Options of the direct share link are enable.
How to reproduce
embedded.geostory.mov
Expected Result
The URL of the embedded geostory does not contain the scroll position
Other useful information
The text was updated successfully, but these errors were encountered: