Skip to content

Commit 40ebb88

Browse files
committed
Use 'nested_statement' from Conditionals, add <ins> markers.
1 parent a869ffd commit 40ebb88

2 files changed

Lines changed: 18 additions & 16 deletions

File tree

css-variables/Overview.html

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -36,14 +36,14 @@
3636

3737
<h1>CSS Variables Module Level 1</h1>
3838

39-
<h2 class="no-num no-toc" id=longstatus-date>Editor's Draft 24 September
39+
<h2 class="no-num no-toc" id=longstatus-date>Editor's Draft 28 September
4040
2011</h2>
4141

4242
<dl>
4343
<dt>This version:
4444

4545
<dd><a
46-
href="http://www.w3.org/TR/2011/ED-css-variables-20110924/">http://dev.w3.org/csswg/css-variables/</a>
46+
href="http://www.w3.org/TR/2011/ED-css-variables-20110928/">http://dev.w3.org/csswg/css-variables/</a>
4747
<!--<dt>Latest version:
4848
<dd><a href="http://www.w3.org/TR/css-variables/">http://www.w3.org/TR/css-variables/</a>-->
4949

@@ -538,7 +538,7 @@ <h2 id=grammar><span class=secno>5. </span> The Grammar of Variables</h2>
538538
<p>and the grammar by adding:
539539

540540
<pre>
541-
variable_declaration
541+
variable_rule
542542
: VARIABLE_SYM S+ variable_name S+ expr ':';
543543
;</pre>
544544

@@ -548,21 +548,22 @@ <h2 id=grammar><span class=secno>5. </span> The Grammar of Variables</h2>
548548
stylesheet
549549
: [ CHARSET_SYM STRING ';' ]?
550550
[S|CDO|CDC]* [ import [ CDO S* | CDC S* ]* ]*
551-
[ [ ruleset | media | page | variable_declaration ] [ CDO S* | CDC S* ]* ]*
551+
[ [ ruleset | media | page <ins>| variable_rule</ins> ] [ CDO S* | CDC S* ]* ]*
552552
;
553-
media
554-
: MEDIA_SYM S* media_list '{' S* [ ruleset | variable_declaration ]* '}' S*
553+
nested_statement
554+
: ruleset | media | page | font_face_rule | keyframes-rule |
555+
supports_rule | document_rule <ins>| variable_rule</ins>
555556
;
556557
term
557558
: unary_operator?
558559
[ NUMBER S* | PERCENTAGE S* | LENGTH S* | EMS S* | EXS S* | ANGLE S* |
559560
TIME S* | FREQ S* ]
560-
| STRING S* | IDENT S* | URI S* | hexcolor | function | VARIABLE S*
561+
| STRING S* | IDENT S* | URI S* | hexcolor | function <ins>| VARIABLE S*</ins>
561562
;</pre>
562563

563-
<p class=issue>This should instead amend the "nested_statement" production
564-
in the Conditionals spec. Should I do so and reference Conditionals, or
565-
are we going to move that to a Syntax module?
564+
<p class=issue>This uses the &lsquo;<code
565+
class=css>nested_statement</code>&rsquo; production from Conditionals.
566+
That should make its way to a proper draft, like a new release of Syntax.
566567

567568
<p class=issue>Should variables be usable elsewhere, like in the value of a
568569
MQ?

css-variables/Overview.src.html

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -281,7 +281,7 @@ <h2 id='grammar'>
281281
<p>and the grammar by adding:</p>
282282

283283
<pre>
284-
variable_declaration
284+
variable_rule
285285
: VARIABLE_SYM S+ variable_name S+ expr ':';
286286
;</pre>
287287

@@ -291,19 +291,20 @@ <h2 id='grammar'>
291291
stylesheet
292292
: [ CHARSET_SYM STRING ';' ]?
293293
[S|CDO|CDC]* [ import [ CDO S* | CDC S* ]* ]*
294-
[ [ ruleset | media | page | variable_declaration ] [ CDO S* | CDC S* ]* ]*
294+
[ [ ruleset | media | page <ins>| variable_rule</ins> ] [ CDO S* | CDC S* ]* ]*
295295
;
296-
media
297-
: MEDIA_SYM S* media_list '{' S* [ ruleset | variable_declaration ]* '}' S*
296+
nested_statement
297+
: ruleset | media | page | font_face_rule | keyframes-rule |
298+
supports_rule | document_rule <ins>| variable_rule</ins>
298299
;
299300
term
300301
: unary_operator?
301302
[ NUMBER S* | PERCENTAGE S* | LENGTH S* | EMS S* | EXS S* | ANGLE S* |
302303
TIME S* | FREQ S* ]
303-
| STRING S* | IDENT S* | URI S* | hexcolor | function | VARIABLE S*
304+
| STRING S* | IDENT S* | URI S* | hexcolor | function <ins>| VARIABLE S*</ins>
304305
;</pre>
305306

306-
<p class='issue'>This should instead amend the "nested_statement" production in the Conditionals spec. Should I do so and reference Conditionals, or are we going to move that to a Syntax module?</p>
307+
<p class='issue'>This uses the 'nested_statement' production from Conditionals. That should make its way to a proper draft, like a new release of Syntax.</p>
307308

308309
<p class='issue'>Should variables be usable elsewhere, like in the value of a MQ?</p>
309310

0 commit comments

Comments
 (0)