Skip to content

Commit 9ceafa7

Browse files
committed
Merge pull request #17 from frivoal/florian/css-ui-warnings
[css-ui] Fix all current bikeshed warnings
2 parents 9550767 + bababbd commit 9ceafa7

File tree

2 files changed

+47
-73
lines changed

2 files changed

+47
-73
lines changed

css-ui/Overview.bs

Lines changed: 21 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,7 @@ Link Defaults: css21 (property) max-width
2828
Link Defaults: css21 (property) max-height
2929
Ignored Terms: outline-top
3030
Ignored Terms: outline-left
31-
Ignored Terms: nav-index
3231
Ignored Terms: ime-mode
33-
Ignored Terms: icon
3432
At Risk: 'box-sizing' property value: padding-box
3533
At Risk: 'caret-color' property
3634
At Risk: 'text-overflow' property value: <<string>>
@@ -152,7 +150,7 @@ cannot be negative</a> ([[!CSS21]], section 10.2),
152150
this computation is floored at 0.
153151
Used values, as exposed for instance through {{getComputedStyle()}}, also refer to the padding box.
154152
</dd>
155-
<dt>border-box</dt>
153+
<dt><dfn>border-box</dfn></dt>
156154
<dd>Length and percentages values for width and height (and respective min/max properties)
157155
on this element determine the border box of the element.
158156
That is, any padding or border specified on the element
@@ -172,7 +170,7 @@ by legacy HTML user agents for replaced elements and input elements.
172170
</dl>
173171

174172
Note: In contrast to the length and percentage values,
175-
the ''auto'' value of the 'width' and 'height' properties
173+
the ''width/auto'' value of the 'width' and 'height' properties
176174
(as well as other keyword values introduced by later specifications,
177175
unless otherwise specified)
178176
is not influenced by the 'box-sizing' property,
@@ -191,7 +189,7 @@ The following terms, whose definitions vary based on the computed value of 'box-
191189
The <a href="http://www.w3.org/TR/CSS21/visudet.html">Visual formatting model details</a> of [[!CSS21]] are written assuming ''box-sizing: content-box''. The following disambiguations are made to clarify the behavior for all values of 'box-sizing':
192190

193191
<ol>
194-
<li>In <a href="http://www.w3.org/TR/CSS21/visudet.html#blockwidth">10.3.3</a>, the second <q>width</q> in the following phrase is to be interpreted as <a spec="css21">content width</a>: <q>If 'width' is not ''auto'' and 'border-left-width' + 'padding-left' + 'width' + [...]</q>
192+
<li>In <a href="http://www.w3.org/TR/CSS21/visudet.html#blockwidth">10.3.3</a>, the second <q>width</q> in the following phrase is to be interpreted as <a spec="css21">content width</a>: <q>If 'width' is not ''width/auto'' and 'border-left-width' + 'padding-left' + 'width' + [...]</q>
195193

196194
<li>In <a href="http://www.w3.org/TR/CSS21/visudet.html#abs-non-replaced-width">10.3.7</a>, <q>width</q> is to be interpreted as <a spec="css21">content width</a> in the following equation: <q>'left' + 'margin-left' + 'border-left-width' + 'padding-left' + 'width' + [...]</q>
197195

@@ -304,7 +302,7 @@ The rendering of applying transforms to outlines is left explicitly undefined in
304302
<tr><th>Inherited:</th><td>no</td></tr>
305303
<tr><th>Percentages:</th><td>N/A</td></tr>
306304
<tr><th>Media:</th><td>visual</td></tr>
307-
<tr><th>Computed&nbsp;value:</th><td>absolute length; ''0'' if the outline style is ''none''.</td></tr>
305+
<tr><th>Computed&nbsp;value:</th><td>absolute length; ''0'' if the outline style is ''outline-width/none''.</td></tr>
308306
<tr><th>Animatable:</th><td>as <a href="http://dev.w3.org/csswg/css3-transitions/#animtype-length">length</a></td></tr>
309307
</tbody></table>
310308

@@ -376,11 +374,11 @@ the same values as
376374
([[!CSS3BG]],
377375
<a href="http://www.w3.org/TR/css3-background/#the-border-style">Section 4.2</a>),
378376
except that
379-
''hidden'' is not a legal outline style.
377+
<span class=css>hidden</span> is not a legal outline style.
380378
In addition, in CSS3,
381379
'outline-style'
382-
accepts the value ''auto''.
383-
The ''auto'' value permits the user agent
380+
accepts the value ''outline-style/auto''.
381+
The ''outline-style/auto'' value permits the user agent
384382
to render a custom outline style,
385383
typically a style which is either a user interface default for the platform,
386384
or perhaps a style that is richer
@@ -390,8 +388,8 @@ that appears to glow.
390388
As such, this specification does not define how the
391389
'outline-color'
392390
is incorporated or used (if at all) when rendering
393-
''auto'' style outlines.
394-
User agents may treat ''auto'' as
391+
''outline-style/auto'' style outlines.
392+
User agents may treat ''outline-style/auto'' as
395393
''solid''.
396394

397395
The 'outline-color' property
@@ -406,7 +404,7 @@ on platforms that do not support color inversion of the pixels on the screen.
406404
If the UA does not support the ''invert'' value
407405
then it must reject that value at parse-time, and
408406
the initial value of the 'outline-color' property
409-
is the ''currentColor'' [[!CSS3COLOR]] keyword.
407+
is the ''color/currentColor'' [[!CSS3COLOR]] keyword.
410408

411409
The 'outline' property is a shorthand property,
412410
and sets all three of 'outline-style',
@@ -540,7 +538,7 @@ to allow the user to adjust only the height of the element.</dd>
540538

541539
Currently it is possible to control the appearance of the scrolling mechanism (if any)
542540
on an element using the 'overflow' property
543-
(e.g. '<code class="lang-css">overflow: scroll</code>' vs. '<code class="lang-css">overflow: hidden</code>' etc.).
541+
(e.g. <code class="lang-css">overflow: scroll</code> vs. <code class="lang-css">overflow: hidden</code> etc.).
544542
The purpose of the 'resize' property
545543
is to also allow control over the appearance and function of the resizing mechanism
546544
(e.g. a resize box or widget) on the element.
@@ -563,7 +561,7 @@ When an element is resized by the user,
563561
the user agent sets
564562
the 'width' and 'height' properties
565563
to px unit length values of the size indicated by the user,
566-
in the element’s <a spec="css-style-attr">style attribute</a> DOM,
564+
in the element’s <a href="http://www.w3.org/TR/css-style-attr/#style-attribute">style attribute</a> DOM,
567565
replacing existing property declaration(s) if any,
568566
without ''!important'' if any.
569567

@@ -588,11 +586,11 @@ since
588586

589587
Note: There may be situations where user attempts to resize an element
590588
to appear to be overriden or ignored, e.g. because of ''!important'' cascading declarations that supersede
591-
that element’s <a spec="css-style-attr">style attribute</a>
589+
that element’s <a href="http://www.w3.org/TR/css-style-attr/#style-attribute">style attribute</a>
592590
'width' and 'height' properties in the DOM.
593591

594592
Changes to the computed value of an element's 'resize' property
595-
do not reset changes to the <a spec="css-style-attr">style attribute</a> made due to
593+
do not reset changes to the <a href="http://www.w3.org/TR/css-style-attr/#style-attribute">style attribute</a> made due to
596594
user resizing of that element.
597595

598596
<div class="example"><p style="display:none">Example(s):
@@ -659,7 +657,7 @@ that has 'overflow'
659657
other than ''visible''.
660658

661659
Text can overflow for example when it is prevented from wrapping
662-
(e.g. due to '<code class="lang-css">white-space: nowrap</code>'
660+
(e.g. due to <code class="lang-css">white-space: nowrap</code>
663661
or a single word is too long to fit).
664662
Values have the following meanings:
665663

@@ -841,8 +839,8 @@ WON'T ELLIPSE.</div>
841839
Note: the side of the line that the ellipsis is placed depends on the 'direction' of the block.
842840
E.g. an overflow hidden right-to-left
843841
(<code class="lang-css">direction: rtl</code>)
844-
block clips inline content on the <a>left</a> side,
845-
thus would place a text-overflow ellipsis on the <a>left</a>
842+
block clips inline content on the <a spec=css-writing-modes-3>left</a> side,
843+
thus would place a text-overflow ellipsis on the <a spec=css-writing-modes-3>left</a>
846844
to represent that clipped content.
847845

848846
<!-- insert RTL example diagram here to illustrate note. -->
@@ -1155,7 +1153,7 @@ often rendered as a magnifying glass with a "+" or "-" in the center of the glas
11551153
</dd>
11561154
</dl>
11571155

1158-
The UA may treat unsupported values as ''auto''.
1156+
The UA may treat unsupported values as ''cursor/auto''.
11591157
E.g. on platforms that do not have a concept of a ''context-menu'' cursor,
11601158
the UA may render ''default'' or whatever is appropriate.
11611159

@@ -1211,7 +1209,7 @@ boxes are generated for it and its cursor is used for the canvas.
12111209
<tr><th>Inherited:</th><td>no</td></tr>
12121210
<tr><th>Percentages:</th><td>N/A</td></tr>
12131211
<tr><th>Media:</th><td>interactive</td></tr>
1214-
<tr><th>Computed&nbsp;value:</th><td>The computed value for ''auto'' is ''currentColor''. For <<color>> values, the computed value is as defined for the [[!CSS3COLOR]] 'color' property.</td></tr>
1212+
<tr><th>Computed&nbsp;value:</th><td>The computed value for ''caret-color/auto'' is ''color/currentColor''. For <<color>> values, the computed value is as defined for the [[!CSS3COLOR]] 'color' property.</td></tr>
12151213
<tr><th>Animatable:</th><td>no</td></tr>
12161214
</tbody></table>
12171215

@@ -1301,7 +1299,7 @@ For accessibility and user convenience,
13011299
user agents should allow configuration of which keys on a keyboard are used for directional navigation.
13021300

13031301
<div class="example">
1304-
<h5>Example: positioned buttons</h5>
1302+
<h5 id=example-positioned-buttons>Example: positioned buttons</h5>
13051303

13061304
Here is an example of buttons positioned in a diamond shape
13071305
whose directional focus navigation is set in such a way
@@ -1342,7 +1340,7 @@ it is still important to ensure focus navigation behaviors which relate reasonab
13421340
</div>
13431341

13441342
<div class="example">
1345-
<h5>Example: moving focus to inside a frame</h5>
1343+
<h5 id=example-moving-focus-to-inside-a-frame>Example: moving focus to inside a frame</h5>
13461344

13471345
Moving the focus to an element in a specific frame requires both the element's id and the frame's name.
13481346

0 commit comments

Comments
 (0)