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
    522
    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,597
    Vote Rating
    54
    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
    119
    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,966
    Vote Rating
    15
    hendricd has a spectacular aura about hendricd has a spectacular aura about

      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