effects slide_in/out sometimes break resizing

Bug #728457 reported by Brad Crittenden
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
High
Unassigned

Bug Description

After using slide_in the height of the node is set to 0. Subsequently using slide_in the scrollHeight must be obtained. In some situations the node incorrectly reports a scrollHeight of 0 and thus cannot open properly.

A solution was suggested by Thomas Herve in
http://paste.ubuntu.com/574976/

The following needs to be added to incorporated into add_slide_state_events:

For slide_in:

    anim.on("start", function() {
        node.setStyles({
            visibility: 'visible'
        });
    });
    anim.on("end", function() {
        node.setStyles({
            height: 0,
            visibility: 'hidden'
        });
    });

For slide_out:

    anim.on("start", function() {
        // Set the node to 'visible' for the beginning of the animation.
        node.setStyles({
            visibility: 'visible'
        });
    });
    anim.on("end", function() {
        // Change the height to auto when the animation completes.
        node.setStyles({
            height: 'auto'
        });
    });

Brad Crittenden (bac)
Changed in lazr-js:
assignee: nobody → Launchpad Yellow Squad (yellow)
status: New → Confirmed
Revision history for this message
Robert Collins (lifeless) wrote :

Updated metadata per https://dev.launchpad.net/BugTriage#How to triage

Changed in lazr-js:
importance: Undecided → High
status: Confirmed → Triaged
affects: lazr-js → launchpad
William Grant (wgrant)
Changed in launchpad:
assignee: Yellow Squad (yellow) → nobody
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.