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

Bail out location.hash setter when set to the current hash #7473

Merged
merged 2 commits into from
Jan 12, 2022

Conversation

domenic
Copy link
Member

@domenic domenic commented Jan 7, 2022

This adds a special case which is necessary for compatibility with deployed content, and implemented in 2/3 engines. Closes #7386.

(See WHATWG Working Mode: Changes for more details.)


/history.html ( diff )

This adds a special case which is necessary for compatibility with deployed content, and implemented in 2/3 engines. Closes #7386.
@domenic domenic added normative change topic: navigation impacts documentation Used by documentation communities, such as MDN, to track changes that impact documentation labels Jan 7, 2022
@domenic domenic requested a review from annevk January 7, 2022 17:03
source Show resolved Hide resolved
@domenic domenic merged commit 3110af8 into main Jan 12, 2022
@domenic domenic deleted the location-hash-setter branch January 12, 2022 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impacts documentation Used by documentation communities, such as MDN, to track changes that impact documentation normative change topic: navigation
Development

Successfully merging this pull request may close these issues.

What should happen when setting location.hash = location.hash?
2 participants