Hybrid View

    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,074
    Vote Rating
    854
    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,505
    Vote Rating
    52
    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,505
    Vote Rating
    52
    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