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

Fix double "update the navigation API" for fragment navs #9990

Merged
merged 1 commit into from
Jan 9, 2024

Conversation

domenic
Copy link
Member

@domenic domenic commented Dec 13, 2023

Closes #9826. da3a62e made it easy to pass along the correct navigationType, and thus trigger the proper path through "update the navigation API entries for a same-document navigation".

Also fixes a typographical issue from da3a62e.

/cc @ADKaster; if you have time to take a look that would be much appreciated!


/browsing-the-web.html ( diff )

Closes #9826. da3a62e made it easy to pass along the correct navigationType, and thus trigger the proper path through "update the navigation API entries for a same-document navigation".

Also fixes a typographical issue from da3a62e.
@domenic domenic force-pushed the fix-double-navigation-api-update branch from 7bca3cd to 304b15b Compare December 13, 2023 06:11
@domenic domenic merged commit b4e2ade into main Jan 9, 2024
2 checks passed
@domenic domenic deleted the fix-double-navigation-api-update branch January 9, 2024 04:09
mobounya pushed a commit to mobounya/ladybird that referenced this pull request Jul 21, 2024
Update 'update_for_history_step_application' to meet some of the specs
introduced in whatwg/html#9856 and in
whatwg/html#9990
mobounya pushed a commit to mobounya/ladybird that referenced this pull request Jul 21, 2024
Update 'update_for_history_step_application' to meet some of the specs
introduced in whatwg/html#9856 and in
whatwg/html#9990
mobounya pushed a commit to mobounya/ladybird that referenced this pull request Jul 21, 2024
Update 'update_for_history_step_application' to meet some of the specs
introduced in whatwg/html#9856 and in
whatwg/html#9990
mobounya pushed a commit to mobounya/ladybird that referenced this pull request Jul 21, 2024
Update 'update_for_history_step_application' to meet some of the specs
introduced in whatwg/html#9856 and in
whatwg/html#9990
mobounya pushed a commit to mobounya/ladybird that referenced this pull request Jul 21, 2024
Update 'update_for_history_step_application' to meet some of the specs
introduced in whatwg/html#9856 and in
whatwg/html#9990
mobounya pushed a commit to mobounya/ladybird that referenced this pull request Jul 22, 2024
Update 'update_for_history_step_application' to meet some of the specs
introduced in whatwg/html#9856 and in
whatwg/html#9990
kalenikaliaksandr pushed a commit to LadybirdBrowser/ladybird that referenced this pull request Jul 22, 2024
Update 'update_for_history_step_application' to meet some of the specs
introduced in whatwg/html#9856 and in
whatwg/html#9990
circl-lastname pushed a commit to circl-lastname/serenity that referenced this pull request Jul 22, 2024
Update 'update_for_history_step_application' to meet some of the specs
introduced in whatwg/html#9856 and in
whatwg/html#9990

(cherry picked from commit 2497f43989af3efa695d0d33de59001cb87bacb6)
nico pushed a commit to SerenityOS/serenity that referenced this pull request Jul 22, 2024
Update 'update_for_history_step_application' to meet some of the specs
introduced in whatwg/html#9856 and in
whatwg/html#9990

(cherry picked from commit 2497f43989af3efa695d0d33de59001cb87bacb6)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

Navigation: navigate to a fragment updates the navigation api for same-document navigation twice.
2 participants