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
    515
    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

    [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,507
    Vote Rating
    56
    Animal has a spectacular aura about Animal has a spectacular aura about Animal has a spectacular aura about

      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
    94
    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 - Community Support Team hendricd's Avatar
    Join Date
    Aug 2007
    Location
    Long Island, NY USA
    Posts
    5,962
    Vote Rating
    10
    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