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

cleanup <pre class=anchors> and use <pre class="link-defaults"> as appropriate #1489

Open
equalsJeffH opened this issue Sep 30, 2020 · 0 comments · May be fixed by #2111
Open

cleanup <pre class=anchors> and use <pre class="link-defaults"> as appropriate #1489

equalsJeffH opened this issue Sep 30, 2020 · 0 comments · May be fixed by #2111
Assignees
Labels
priority:low @Risk Items that are at risk for L3 type:editorial

Comments

@equalsJeffH
Copy link
Contributor

per @tabatkins:

Tab: If you're using <pre class=anchors>, that's very likely wrong. That's for providing all the info of an autolink, when a spec doesn't provide autolinks at all. There's a lot of copypasta using that, dating from when HTML wasn't marked up reasonably.

me: so, we're using both <pre class="link-defaults"> (some) and (mostly) <pre class=anchors>

Tab: Okay, so your "anchors" block is very likely completely unnecessary. Try removing it and seeing what still fails. If entire specs are missing, check if they're in Shepherd at all, or if I need to add them.

me: what if a term is <dfn>'d in multiple specs that are in Shepherd ?

Tab: That's just a normal ambiguous link, and <pre class="link-defaults"> can handle that to specify which spec you want it from. (Or you can do it at point-of-link with the spec attribute on the autolink.)

@equalsJeffH equalsJeffH added type:editorial priority:low stat:puntable Issue or PR that is candidate to move to a later milestone labels Sep 30, 2020
@nadalin nadalin added this to the L2-WD-04 Final milestone Oct 7, 2020
@equalsJeffH equalsJeffH modified the milestones: L2-WD-04 Final, L2-CR Nov 11, 2020
@nadalin nadalin modified the milestones: L2-CR, L3-WD-01 Nov 18, 2020
@nadalin nadalin removed the stat:puntable Issue or PR that is candidate to move to a later milestone label Nov 18, 2020
@nadalin nadalin added the @Risk Items that are at risk for L3 label Jun 27, 2023
@plehegar plehegar modified the milestones: L3-WD-01, L3-WD-02 Oct 4, 2023
@emlun emlun linked a pull request Sep 24, 2024 that will close this issue
@timcappalli timcappalli self-assigned this Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:low @Risk Items that are at risk for L3 type:editorial
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants