Assigning children in visible trap does not fire child visible traps

Bug #268627 reported by Charles Goodwin
2
Affects Status Importance Assigned to Milestone
Vexi
Fix Released
Medium
Charles Goodwin

Bug Description

<ui:box display="false">
    <ui:box id="a" /> // not visible
    <ui:box id="b" /> // not visible
</ui:box>
<ui:box id="c" /> // visible

$a.visible ++= function(v) {
    cascade = v;
    vexi.log.info("a now visible? "+v);
    $a[0] = $b;
}

$b.visible ++= function(v) {
    cascade = v;
    vexi.log.info("b now visible? "+v);
}

// only outputs for $a, not $b
$c[0] = $a;

The code that handles surface/visible puts needs rewriting to make use of a Callable that:
- depth-first call preTriggerTraps for surface/visible
- assigns box to new parent
- depth-first call postTriggerTraps

Changed in vexi:
importance: Undecided → Medium
milestone: none → 3.0-future
assignee: nobody → charlesgoodwin
Revision history for this message
Charles Goodwin (charlesgoodwin) wrote :

Fixed in r3164

Changed in vexi:
status: New → Fix Committed
Changed in vexi:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.