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

problems with @ident of <remarks> #2681

Open
sydb opened this issue Feb 12, 2025 · 0 comments
Open

problems with @ident of <remarks> #2681

sydb opened this issue Feb 12, 2025 · 0 comments

Comments

@sydb
Copy link
Member

sydb commented Feb 12, 2025

The <remarks> element has its own @ident defined as teidata.text with the description “specifies the remark concerned.”.
At first blush, almost everything about that seems wrong. (After thinking about it, only the definition and description seem wrong.)

  • Why does <remarks> get an @ident at all? It is not processed by the Stylesheets. (The answer is that we hope to change the Stylesheets so that they are properly processed, someday.)
  • Given that it has an @ident, why not get it from being a member of att.identified? (The answer is, I would guess, that we did not want @predeclare, @module, or @validUntil, and thought it better to add it directly than to add it to att.identified and then delete those attributes.)
  • Why is it defined as teidata.text rather than teidata.name? (I think this is just a corrigible error that should be fixed ASAR.)
  • The description is wrong. At the least it should be “identifies the remark concerned”; but to be more like att.identified probably “supplies the identifier by which these remarks may be referenced.”
@sydb sydb added this to the Guidelines 4.10.0 milestone Feb 12, 2025
sydb added a commit that referenced this issue Feb 12, 2025
the concerns voiced in #2681
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant