Specs: note one of the big problems with the current style system: that rule order isn't maintained

sky/specs/style.md

 Sky Style Language
 ==================
 
 Planed changes
 --------------
 
 Add //-to-end-of-line comments to be consistent with the script
 language.
 
 
@@ skipping 238 matching lines @@
 -------------
 
 partial class Element {
   readonly attribute StyleDeclarationList style;
 }
 
 class StyleDeclarationList {
   constructor ();
   void add(StyleDeclaration styles, String? pseudoElement = null); // O(1) // in debug mode, throws if the dictionary has any properties that aren't registered
   void remove(StyleDeclaration styles, String? pseudoElement = null); // O(N) in number of declarations
+  // TODO(ianh): Need to support inserting rules preserving order somehow
   Array<StyleDeclaration> getDeclarations(String? pseudoElement = null); // O(N) in number of declarations
 }
 
 typedef StyleDeclaration Dictionary<ParsedValue>;
 
 
 Rule Matching
 -------------
 
 partial class StyleElement {
   Array<Rule> getRules(); // O(N) in rules
 }
 
 class Rule {
   constructor ();
   attribute SelectorQuery selector; // O(1)
   attribute String? pseudoElement; // O(1)
   attribute StyleDeclaration styles; // O(1)
 }
 
 Each frame, at some defined point relative to requestAnimationFrame():
  - If a rule starts applying to an element, sky:core calls thatElement.style.add(rule.styles, rule.pseudoElement);
  - If a rule stops applying to an element, sky:core calls thatElement.style.remove(rule.styles, rule.pseudoElement);
 
+TODO(ianh): fix the above so that rule order is maintained
+
 
 Cascade
 -------
 
 For each Element, the StyleDeclarationList is conceptually flattened
 so that only the last declaration mentioning a property is left.
 
 Create the flattened render tree as a tree of StyleNode objects
 (described below). For each one, run the equivalent of the following
 code:
@@ skipping 213 matching lines @@
 * ``error``
   This adds to itself the declaration with value:
   ``{ display: { value: sky.ErrorLayoutManager } }``
 
 The ``div`` element doesn't have any default styles.
 
 These declarations are all shared between all the elements (so e.g. if
 you reach in and change the declaration that was added to a ``title``
 element, you're going to change the styles of all the other
 default-hidden elements).