Skip to content

[Editorial] 'policy' is a local term, not global.#812

Merged
mikewest merged 2 commits into
mainfrom
policy
Apr 19, 2026
Merged

[Editorial] 'policy' is a local term, not global.#812
mikewest merged 2 commits into
mainfrom
policy

Conversation

@mikewest
Copy link
Copy Markdown
Member

@mikewest mikewest commented Apr 16, 2026

@antosart WDYT of this? In short, we're linking to the wrong term as for because I misunderstood what Bikeshed did with local-lt.


Preview | Diff

@antosart
Copy link
Copy Markdown
Member

This seems to break all internal links. E.g. on line 486 [=policy/disposition=] doesn't resolve anymore to the local spec, while [=content security policy object/disposition=] does.

It's not really clear to me how local-lt behaves. Maybe it's easiest to just give up and use the long link everywhere?

@mikewest
Copy link
Copy Markdown
Member Author

That surprises me. :/ @tabatkins might have insight into the expected behavior, but I'll just use the long name for the moment (which looks like it produces the expected output).

@tabatkins
Copy link
Copy Markdown
Member

Note that the for value on dfns/links is purely an opaque namespacing mechanism. It has no inherent connection to any other definition, except by convention. (IDL attribute/method definitions, for example, expect their for values to be the name of the interface, etc.) If a term like "content security policy object" has some associated definitions that you want to link together (and hide from the global namespace), you can do so with whatever for value you desire; Bikeshed does not care whether there is an actual dfn with that term, "local" or not.

@mikewest
Copy link
Copy Markdown
Member Author

Ok. I'll land this, and rethink the way I'm using lt here and elsewhere... :/

@mikewest mikewest merged commit 3e24aea into main Apr 19, 2026
2 checks passed
github-actions Bot added a commit that referenced this pull request Apr 19, 2026
SHA: 3e24aea
Reason: push, by mikewest

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants