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

Rename the document element from "guide" to "devbook" #352

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ulm
Copy link
Member

@ulm ulm commented Oct 24, 2024

This has bugged me since quite some time, mainly because nXML mode in Emacs (or rather, rng-loc) locates the schema based on the XML document element. So, when looking at old documentation, it would see <guide>, load devbook.rnc, and would end up highlighting supposedly bad syntax.

Renaming the document element to devbook will avoid these problems, because guide can then be dropped from nXML mode's schema locating, and the vacuous schema will be used for GuideXML documents (which is not a problem when looking at them).

While at it, rename <contentsTree/> to <contents/> because it is the only element whose name is in camelCase. (No strong opinion about the name here, contents is what is used by Texinfo. So please speak up if you prefer tableofcontents (as in LaTeX), toc (as in Mediawiki), or even childlist (used in the old RST Devmanual).)

Timeline: If there are no objections, I would start with a transitional release of app-emacs/nxml-gentoo-schemas that would accept both the old guide and the new devbook element. The Devmanual would be updated some time after stabilisation of that version.

The "guide" document element is ambiguous with the previously used
GuideXML format. Now that DevBook XML has a schema, this confuses
Emacs nXML mode (which uses the document element to locate a schema),
e.g. when visiting old GuideXML documentation files.

Signed-off-by: Ulrich Müller <[email protected]>
For consistency; this was the only element using camelCase.

Signed-off-by: Ulrich Müller <[email protected]>
Copy link
Member

@thesamesam thesamesam left a comment

Choose a reason for hiding this comment

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

This sounds OK. I trust you on the transition mechanics.

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.

2 participants