[w3c/manifest] Processing steps for id now removes the fragment (PR #1122)

Closes #1121

This change (choose at least one, delete ones that don't apply):

* Adds new normative recommendations or optional items

[Note: This technically changes normative behaviour, but does not break implementations. Any implementation that keeps the fragment and strips it out when comparing ids will behave the same as this. The new behaviour just encourages implementations to keep their local data model clean.]

Commit message:

The canonical id no longer includes the fragment. This was already effectively true (as all comparisons would exclude fragments), but now it makes it true in the processed manifest's canonical id.

Removed or made optional the exclude fragments in the comparisons. Added a lengthy note as to why user agents may still wish to exclude fragments when comparing against potentially old database entries.

Person merging, please make sure that commits are squashed with one of the following as a commit message prefix:

* chore:
* editorial:
* BREAKING CHANGE:
* And use none if it's a normative change


<!--
    This comment and the below content is programmatically generated.
    You may add a comma-separated list of anchors you'd like a
    direct link to below (e.g. #idl-serializers, #idl-sequence):

    Don't remove this comment or modify anything below this line.
    If you don't want a preview generated for this pull request,
    just replace the whole of this comment's content by "no preview"
    and remove what's below.
-->
***
<a href="https://pr-preview.s3.amazonaws.com/mgiuca/manifest/pull/1122.html" title="Last updated on May 6, 2024, 4:46 AM UTC (70792c0)">Preview</a> | <a href="https://pr-preview.s3.amazonaws.com/w3c/manifest/1122/3aab3be...mgiuca:70792c0.html" title="Last updated on May 6, 2024, 4:46 AM UTC (70792c0)">Diff</a>
You can view, comment on, or merge this pull request online at:

  https://github.com/w3c/manifest/pull/1122

-- Commit Summary --

  * (unrelated) more formal language to set scope's query and fragment to null.
  * id processing: Set fragment to null in the processor.
  * Do not exclude fragments when comparing the manifest id with the document manifest id.
  * Properly link to exclude fragments.
  * Added note about why we still exclude fragments when comparing to the app database.
  * Rewrote the exclude fragments note, and made exclude fragments optional.

-- File Changes --

    M index.html (51)

-- Patch Links --

https://github.com/w3c/manifest/pull/1122.patch
https://github.com/w3c/manifest/pull/1122.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/w3c/manifest/pull/1122
You are receiving this because you are subscribed to this thread.

Message ID: <w3c/manifest/pull/1122@github.com>