search

stunn / jssketch

1 stars 0 forks source link

Sort out the Doctype mess #4

Closed mattlunn closed 11 years ago

mattlunn commented 11 years ago

Object or array?

One or the other, Jeez.

mattlunn commented 11 years ago

This was fixed in the above commit. Although the solution is inconsistent with the CSS and JS, this is because both are wrapped in a manager. Introducing a manager for doctypes seemed stupid, and protecting against duplicate IDs and having O(1) lookup by ID was important (which wouldn't exist with array-with-elements-with-id-properties approach).