-
-
Notifications
You must be signed in to change notification settings - Fork 389
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
More flexible title id generation #4712
Comments
I think the W3C pub rules checker needs id=“title” iirc. |
And yeah, that’s weird if it’s not detecting the collision. Is that second title explicitly set to id=title? |
Yeah, the document is explicitly setting the id: <section id="title"><h4>title</h4> Respec won't override author's id. You will need to set it to something else (otherwise w3c pub rules will also break) |
Ok I did not know that. Do we know why they need it? As a first solution we have removed it from our publications.
Yes since it is a vocabulary term of our information model and it is linked by external documents as well so we cannot change it. |
Funny enough, pubrules checker does not complain with missing id=title |
To be sure, that’s the html validator (not the w3c pub rules). However, let me check about the id. |
Also if it is needed for pubrules check, why not add it only when generating a static file? |
Ok, thanks for also checking. Seems I had misremembered. |
Description of problem
URL to affected spec or repo:
https://w3c.github.io/wot-thing-description/ontology/td
What happened (e.g., it crashed)?: ReSpec always inserts the id title for the document title. However, if there is already a section id with value title in the document, it causes issues with linking to that section.
Expected behavior (e.g., it shouldn't crash): In such cases, respec should display a warning. It can also try to iterate and add
title2
etc. Even better would be a configuration option to set a custom title id.Optional, steps to reproduce:
Note: I am not sure why the title needs an id. Maybe removing that addition of the id can make sense?
The text was updated successfully, but these errors were encountered: