Changes between Version 1 and Version 2 of Ticket #13769, comment 40


Ignore:
Timestamp:
Jun 17, 2012, 11:31:33 PM (7 years ago)
Author:
bill
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #13769, comment 40

    v1 v2  
    1 By popular demand, and since Eugene said it was OK, I was about to apply the patch above.   But I hit problems because dojo/dom-attr.js still depends on dojo/dom-prop.js:
     1By popular demand, and since Eugene said it was OK, I was about to apply the patch above.   But I hit problems because dojo/dom-attr::set() still depends on dojo/dom-prop:set()
    22
    33{{{#!js
     
    77}}}
    88
    9 Seems like the patch is insufficient?   dom-attr --> dom-prop --> dom-construct --> dom-attr.
     9Seems like the patch is insufficient?   dom-attr --> dom-prop--> dom-construct (empty, toDom) --> dom-attr:
     10
     11 - dom-attr::set() calls dom-prop::set()
     12 - dom-prop::set() calls dom-construct::empty() and dom-construct::toDom()
     13 - dom-construct::create() calls dom-attr::set()
     14