-
Notifications
You must be signed in to change notification settings - Fork 23
Improve description of keyword aliases using @protected
#272
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
Conversation
…e to the grammar. For #246.
… id-map until that is addressed in the algorithm.
a056cd5
to
eaa444e
Compare
@@ -12050,9 +12057,14 @@ <h2>Context Definitions</h2> | |||
<code>@container</code>, | |||
<code class="changed">@context</code>, | |||
<code class="changed">@prefix</code>, | |||
<code class="changed">@propagate</code>, or |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did remove @propagate
from this list earlier (b4dae4d), because I don't see anything the API where it is expected in an expanded term definition...
Am I missing something?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, I you’re right, only a context definition can contain @propagate
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, I you’re right, only a context definition can contain @propagate
.
@@ -12110,13 +12122,21 @@ <h2>Context Definitions</h2> | |||
<p class="changed">If an <a>expanded term definition</a> has an <code>@context</code> <a>entry</a>, | |||
it MUST be a valid <code>context definition</code>.</p> | |||
|
|||
<p class="changed">If a <a>context</a> contains the <code>@import</code> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This whole paragraph is a copy of the one line 12017... Was the intent to replace "context" by "expanded term definition"?? That would be strange as @import
can not appear in an expanded term definition...
<p class="changed">If the <a>expanded term definition</a> contains the <code>@nest</code> | ||
<a>keyword</a>, its value MUST be either <code>@nest</code>, or a term | ||
which expands to <code>@nest</code>.</p> | ||
|
||
<p class="changed">If the <a>expanded term definition</a> contains the <code>@prefix</code> | ||
<a>keyword</a>, its value MUST be <code>true</code> or <code>false</code>.</p> | ||
|
||
<p class="changed">If the <a>expanded term definition</a> contains the <code>@propagate</code> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cf. my comment above, I don't think that an expanded term definition can contain the @propagate
keyword...
and add more to the grammar.
For #246.
Preview | Diff