Skip to content

Commit 187a49d

Browse files
committed
Add some issues.
1 parent 745e199 commit 187a49d

File tree

1 file changed

+21
-3
lines changed

1 file changed

+21
-3
lines changed

css-typed-om/Overview.bs

+21-3
Original file line numberDiff line numberDiff line change
@@ -708,9 +708,27 @@ of {{CSSResourceValue}}.
708708

709709
Mapping of properties to accepted types {#mapping-of-properties-to-accepted-types}
710710
==================================================================================
711-
This section provides a table of which types of {{CSSStyleValue}} a given property can accept.
712-
Note that most, but not all properties take {{CSSKeywordValue}}.
713-
Shorthand properties and values are not supported.
711+
712+
This section defines which types of {{CSSStyleValue}} a given property can accept.
713+
714+
Issue(300): This is pretty incomplete, and incorrect in some places.
715+
For example, 'align-content' can take up to three keywords.
716+
More complex values like 'box-shadow' seem to be punted entirely.
717+
718+
Issue(301): This doesn't handle shorthand properties,
719+
which is required for v1.
720+
(Properties occasionally get upgraded to shorthands;
721+
the failure to handle this was one of the reasons DOM Level 2 Style was sucky.)
722+
723+
Issue(302): How should we handle values that allow reordering?
724+
We'll *return* them in a particular order,
725+
but should we accept them in any?
726+
Is it better to represent them as specialized value types,
727+
so they'll have field names?
728+
729+
Issue(303): How should we handle optionality? List "Foo | PairValue<Foo>", etc?
730+
731+
Issue(304): What does PairValue/etc even mean? They're not documented.
714732

715733
<table class=data>
716734
<thead>

0 commit comments

Comments
 (0)