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

Editorial: attempt to deduplicate the file state #512

Merged
merged 2 commits into from
May 14, 2020

Conversation

annevk
Copy link
Member

@annevk annevk commented May 11, 2020

Maybe helps with #504, though this is much less convincing than #509 I think.

I also noticed there is not good coverage for file base URLs.


Preview | Diff

Maybe helps with #504, though this is much less convincing than #509 I think.

I also noticed there is not good coverage for file base URLs.
<ol>
<li><p><a>Validation error</a>.

<li><p>Set <var>url</var>'s <a for=url>host</a> to null and <var>url</var>'s
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As host is cleared in https://url.spec.whatwg.org/#path-state (though to "", not null) perhaps it's better to keep it here, also in light of #302.

@annevk annevk requested a review from domenic May 13, 2020 14:38
@annevk annevk merged commit cb9c97f into master May 14, 2020
@annevk annevk deleted the annevk/file-state-deduplication branch May 14, 2020 17:37
domenic pushed a commit to jsdom/whatwg-url that referenced this pull request May 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants