[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: The highlight() method
Hi, Itedo noticed that the highlight() method on the Picture interface may be problematic. The prototype is: void highlight(in WebCGMNodeList nodes, in boolean state); The method works for the following script: var list = pic.getAppStructuresByName("listToHighlight"); pic.highlight( list, true ); however, the following code is invalid: var aps = pic.getAppStructureById("apsToHighlight"); pic.highlight( aps, true ); getAppStructureById returns a WebCGMNode, not a WebCGMNodeList, doh! Here are some options that we have: i) leave it like this, fix in a later version (2.1) but that means WebCGM 2.0 users will not be able to highlight a single Application Structures. ii) change getAppStructureById to return a WebCGMNodeList. This however will force script writers to write: var id = pic.getAppStructureById("myID"); if( id.count > 0 ) { id.item(0). etc... } every time they want to access and APS by Id. iii) have two highlight methods. One that is called highlight( takes a WebCGMNode ) and one called highlightMany( takes a WebCGMNodeList). It wouldn't look very good in the spec, but hey, it would work. iv) introduce some sort of WebCGMNode.makeList() method that would return a WebCGMNodeList from a WebCGMNode. This approach would need more thoughts, as it could complicate the 'live' node implementation of our DOM. Thoughts, -- Benoit mailto:benoit@itedo.com
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]