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

Doc.url not updated in case of Ajax navigation #577

Closed
PierreBrisorgueil opened this issue Dec 3, 2020 · 3 comments · Fixed by #584
Closed

Doc.url not updated in case of Ajax navigation #577

PierreBrisorgueil opened this issue Dec 3, 2020 · 3 comments · Fixed by #584
Labels
status/in-progress type/bug Something isn't working
Milestone
0.14.0

Comments

@PierreBrisorgueil
Copy link
Contributor

Hello,

If you click on a link managed with JS like AJAX, WAIT_NAVIGATION failed, we can simply wait for the update thanks to a WAIT_ELEMENT. But in this case, doc.url is not properly updated.
@ziflex ziflex added status/review-needed type/bug Something isn't working labels Dec 3, 2020
@ziflex
Copy link
Member

ziflex commented Dec 3, 2020

hey, thanks for the report.
could you provide a website and query?

@PierreBrisorgueil
Copy link
Contributor Author

PierreBrisorgueil commented Dec 3, 2020 •
edited
Loading

Yep, I have already sent you everything to test the bug in mp on ferret discord 👌

@ziflex ziflex added this to the 0.14.0 milestone Dec 19, 2020
ziflex added a commit that referenced this issue Jan 19, 2021
@ziflex
Merge branch 'master' into feature/#577-spa-routing
e460a01
@ziflex ziflex mentioned this issue Jan 19, 2021
Merged
ziflex added a commit that referenced this issue Feb 16, 2021
@ziflex
Merge branch 'master' into feature/#577-spa-routing
e3e1214
ziflex added a commit that referenced this issue Feb 16, 2021
@ziflex
Feature/#577 spa routing (#584)
ff8c15e 
* Added support of getting URL dynamically
@PierreBrisorgueil
Copy link
Contributor Author

@ziflex Thx ! I will test it soon ! :)

bundleman pushed a commit to bundleman/ferret that referenced this issue Apr 5, 2021
@ziflex @bundleman
Feature/MontFerret#577 spa routing (MontFerret#584)
0902ddf 
* Added support of getting URL dynamically
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
status/in-progress type/bug Something isn't working
Projects
None yet
Milestone
0.14.0
Development

Successfully merging a pull request may close this issue.

Feature/#577 spa routing MontFerret/ferret
2 participants
@ziflex @PierreBrisorgueil