Success! Looks like we've fixed this one. According to our records the fix was applied for EXTJS-7470 in a recent build.
  1. #1
    Sencha User Daniil's Avatar
    Join Date
    Jun 2010
    Location
    Saint-Petersburg, Russia
    Posts
    974
    Vote Rating
    110
    Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all

      0  

    Default [4.1.2] Border layout: expanding collapsed region caused iframe reloading

    [4.1.2] Border layout: expanding collapsed region caused iframe reloading


    REQUIRED INFORMATION

    Ext version tested:
    • Ext 4.1.2
    Browser versions tested against:
    • Chrome
    • FireFox
    • IE9
    Description:
    • Expanding a region of Border layout caused iframe reloading in the center region.
    • Generally, it is a known issue when ifram reloads when moved into another DOM node, but, I think, it should not happen in this case.
    • It doesn't happen on collapsing.
    • collapseMode: "mini" for the region helps to avoid this issue.
    • animCollapse: false doesn't help.
    Steps to reproduce the problem:
    • Collapse the region
    • Expand the region
    The result that was expected:
    • iframe does not reload
    The result that occurs instead:
    • iframe does reload
    Test Case:

    Code:
    <html>
    <head>
        <title>Border iframe reloading</title>
    
        <link type="text/css" rel="stylesheet" href="../resources/css/ext-all.css" />
        
        <script type="text/javascript" src="../ext-all-debug.js"></script>
    
        <script type="text/javascript">
            Ext.onReady(function () {
                Ext.create("Ext.Viewport", {
                    renderTo: Ext.getBody(),
                    layout: "fit",
                    items: [{
                        layout: "border",
                        items: [{
                            width: 200,
                            region: "west",
                            collapsible: true,
                            animCollapse: false,
                            title: "East"
                        }, {
                            region: "center",
                            contentEl: "iframe1",
                            title: "Center"
                        }]
                    }]
                });
            });
        </script>
    </head>
    <body>
        <iframe id="iframe1" src="test.htm" class="x-hidden"></iframe>
    </body>
    </html>
    HELPFUL INFORMATION

    Operating System:
    • Windows 7

  2. #2
    Sencha - Senior Forum Manager mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    37,227
    Vote Rating
    859
    mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute

      0  

    Default


    Thanks for the report.
    Mitchell Simoens @SenchaMitch
    Sencha Inc, Senior Forum Manager
    ________________
    Check out my GitHub, lots of nice things for Ext JS 4 and Sencha Touch 2
    https://github.com/mitchellsimoens

    Think my support is good? Get more personalized support via a support subscription. https://www.sencha.com/store/

    Need more help with your app? Hire Sencha Services services@sencha.com

    Want to learn Sencha Touch 2? Check out Sencha Touch in Action that is in print!

    When posting code, please use BBCode's CODE tags.

  3. #3
    Sencha User
    Join Date
    Sep 2011
    Posts
    4
    Vote Rating
    0
    pelaez87 is on a distinguished road

      0  

    Default


    Hi,

    I have the same problem, is there any advance or a fixed version.
    In any case, when could we have a release with this bug fixed?

  4. #4
    Sencha - Ext JS Dev Team Animal's Avatar
    Join Date
    Mar 2007
    Location
    Notts/Redwood City
    Posts
    30,506
    Vote Rating
    54
    Animal has a spectacular aura about Animal has a spectacular aura about Animal has a spectacular aura about

      1  

    Default


    Try

    Code:
    Ext.override(Ext.layout.container.Border, {
        isValidParent: function() {
            return true;
        }
    });

  5. #5
    Sencha User Daniil's Avatar
    Join Date
    Jun 2010
    Location
    Saint-Petersburg, Russia
    Posts
    974
    Vote Rating
    110
    Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all

      0  

    Default


    Thank you, @Animal! It works well.

    Please clarify is it just a workaround or a candidate to be a permanent fix?
    Ext.NET - ASP.NET for Ext JS
    MVC and WebForms
    Examples | Twitter

  6. #6
    Sencha - Ext JS Dev Team Animal's Avatar
    Join Date
    Mar 2007
    Location
    Notts/Redwood City
    Posts
    30,506
    Vote Rating
    54
    Animal has a spectacular aura about Animal has a spectacular aura about Animal has a spectacular aura about

      0  

    Default


    Just a workaround.

    The fix is to write isValidParent to correctly determine whether the border child item really does need moving somewhere else within the container.

    It's complicated by the extra Splitter Components being automagically inserted between Components, and the extra weirdness of placeholder Components which are what you see as the collapsed "representation". They are all in there, and isValidParent just doesn't do a good job right now.

    I have the ticket open it will be fixed in 4.2

  7. #7
    Sencha User Daniil's Avatar
    Join Date
    Jun 2010
    Location
    Saint-Petersburg, Russia
    Posts
    974
    Vote Rating
    110
    Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all

      0  

    Default


    Thank you for the details!

    I see it has been marked fixed.

    So, am I right to expect the fix in 4.1.3 or, maybe, 4.1.4? Or is 4.2 still actual?
    Ext.NET - ASP.NET for Ext JS
    MVC and WebForms
    Examples | Twitter

  8. #8
    Sencha - Ext JS Dev Team Animal's Avatar
    Join Date
    Mar 2007
    Location
    Notts/Redwood City
    Posts
    30,506
    Vote Rating
    54
    Animal has a spectacular aura about Animal has a spectacular aura about Animal has a spectacular aura about

      0  

    Default


    4.1.3 is out. The code went into the 4.2 branch

  9. #9
    Sencha User Daniil's Avatar
    Join Date
    Jun 2010
    Location
    Saint-Petersburg, Russia
    Posts
    974
    Vote Rating
    110
    Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all Daniil is a name known to all

      0  

    Default


    Ok, thanks!

    If you will have a chance, could you look at this, please?
    http://www.sencha.com/forum/showthread.php?246801

    Just I am interested in your opinion. I know you always have an independent opinion

    (Apologize for a trick. Please just ignore if you wish.)
    Ext.NET - ASP.NET for Ext JS
    MVC and WebForms
    Examples | Twitter

  10. #10
    Sencha User
    Join Date
    Jan 2013
    Posts
    4
    Vote Rating
    0
    slee90 is on a distinguished road

      0  

    Default


    i think this problem still exists in IE 9. if I did panel.getLoader().load(), it throws a bunch of js errors in the URL provided. tested with other versions of IE and browsers, works fine.