Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Touch Premium Member
    Join Date
    Jan 2008
    Location
    Germany
    Posts
    521
    Vote Rating
    0
    Stefan B is on a distinguished road

      0  

    Default [FIXED-129][Ext 3.0.3] Form upload: hidden iframe not hidden in FF

    When submitting a form with a {fileUpload: true} configuration, Ext temporarily creates an iframe, adds an HTML form and performs a standard submit on it.

    The temporary iframe should not be visible, so Ext adds a class "x-hidden" to it. This is done with the following code in Ext.data.Connection#doFormUpload:

    Code:
    Ext.fly(frame).set({
        id: id,
        name: id,
        className: 'x-hidden',
        src: Ext.SSL_SECURE_URL // for IE
    });
    In FF (3.5.5 in my case) setting the class via Element.set({className: "..."}) fails and results in the following HTML, causing the iframe to stay visible during its lifetime:

    Code:
    <iframe id="ext-genXXX" name="ext-genXXX" classname="x-hidden" src="blank.html">
    Tested on IE7 also, no problem there.

    Thanks for looking into this,
    Stefan

  2. #2
    Sencha - Ext JS Dev Team Animal's Avatar
    Join Date
    Mar 2007
    Location
    Notts/Redwood City
    Posts
    30,565
    Vote Rating
    55
    Animal will become famous soon enough Animal will become famous soon enough

      0  

    Default

    Should be

    Code:
    Ext.fly(frame).set({
        id: id,
        name: id,
        className: 'x-hidden',
        src: Ext.SSL_SECURE_URL // for IE
    }, false);

  3. #3
    Sencha - Community Support Team Condor's Avatar
    Join Date
    Mar 2007
    Location
    The Netherlands
    Posts
    24,246
    Vote Rating
    107
    Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of

      0  

    Default

    IMHO it should be:
    Code:
    Ext.fly(frame).set({
        id: id,
        name: id,
        cls: 'x-hidden' // fixed in Core rev. 70
    });
    doc.body.appendChild(frame);
    Ext.fly(frame).set({
        src: Ext.SSL_SECURE_URL // for IE and Opera 10
    });

  4. #4
    Sencha - Services Team hendricd's Avatar
    Join Date
    Aug 2007
    Location
    Long Island, NY USA
    Posts
    5,965
    Vote Rating
    12
    hendricd will become famous soon enough hendricd will become famous soon enough

      0  

    Default

    Fixed in Core-SVN for 3.x.
    "be dom-ready..."
    Doug Hendricks

    Maintaining ux: ManagedIFrame, MIF2 (FAQ, Wiki), ux.Media/Flash, AudioEvents, ux.Chart[Fusion,OFC,amChart], ext-basex.js/$JIT, Documentation Site.


    Got Sencha licensing questions? Find out more here.


Thread Participants: 3