Success! Looks like we've fixed this one. According to our records the fix was applied for EXTJS-5782 in a recent build.
  1. #1
    Sencha User
    Join Date
    Mar 2012
    Posts
    3
    Vote Rating
    0
    kwh is on a distinguished road

      0  

    Default htmleditor in tab panel breaks layout when tab is destroyed

    htmleditor in tab panel breaks layout when tab is destroyed


    REQUIRED INFORMATION

    Ext version tested:
    • Ext 4.1-RC1
    Browser versions tested against:
    • IE9
    • FF11 (firebug 1.9.1 installed)
    • Chrome 17
    DOCTYPE tested against:
    • <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">
    Description:
    • When closing a closable tab containing a htmleditor field, nested or not causes javascript exception.
    • Does not happen if the tab is not the active tab at the time of close.
    • Worked in ext-4.1-b3
    Steps to reproduce the problem:
    • Run attached file (based on Window Layout example)
    The result that was expected:
    • Tab to close cleanly
    The result that occurs instead:
    • Javascript error which breaks entire layout
    Test Case:

    Code:
    /**
     * Copy this into: 
     * extjs-4.1.0-rc1\examples\window\layout.js 
     */
    Ext.require([
        'Ext.tab.*',
        'Ext.window.*',
        'Ext.tip.*',
        'Ext.layout.container.Border'
    ]);
    Ext.onReady(function(){
        var win,
            button = Ext.get('show-btn');
    
        button.on('click', function(){
             if (!win) {
                win = Ext.create('widget.window', {
                    title: 'Layout Window',
                    closable: true,
                    closeAction: 'hide',
                    width: 600,
                    minWidth: 350,
                    height: 350,
                    layout: {
                        type: 'border',
                        padding: 5
                    },
                    items: [{
                        region: 'west',
                        title: 'Navigation',
                        width: 200,
                        split: true,
                        collapsible: true,
                        floatable: false
                    }, {
                        region: 'center',
                        xtype: 'tabpanel',
                        items: [{
                            title: 'Bogus Tab',
                            html: 'Hello world 1'
                        }, {
                            title: 'Another Tab',
                            html: 'Hello world 2'
                        }, {
                            title: 'Closable Tab',
                            html: 'Hello world 3',
                            closable: true
                        },{
                            xtype: 'htmleditor',
                            name: 'closeme',
                            title: 'htmleditor',
                            value: 'test vaule',
                            id: 'html-editor',
                            closable: true,
                        }]
                    }]
                });
            }
            button.dom.disabled = true;
            if (win.isVisible()) {
                win.hide(this, function() {
                    button.dom.disabled = false;
                });
            } else {
                win.show(this, function() {
                    button.dom.disabled = false;
                });
            }
        });
    });

    HELPFUL INFORMATION

    Screenshot or Video:
    • none
    Debugging already done:
    • none
    Possible fix:
    • not provided
    Additional CSS used:
    • Only default ext-all.css used in the window layout example
    Operating System:
    • Win7

  2. #2
    Sencha - Senior Forum Manager mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    37,652
    Vote Rating
    901
    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
    Mar 2012
    Posts
    3
    Vote Rating
    0
    kwh is on a distinguished road

      0  

    Default Has this been fixed in SVN?

    Has this been fixed in SVN?


    I didn't spot this bug under known issues in the RC2 release notes.

  4. #4
    Sencha - Senior Forum Manager mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    37,652
    Vote Rating
    901
    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


    Quote Originally Posted by kwh View Post
    I didn't spot this bug under known issues in the RC2 release notes.
    This issue has been fixed and will be in the next release (GA)
    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.

  5. #5
    Sencha User
    Join Date
    Mar 2012
    Posts
    3
    Vote Rating
    0
    kwh is on a distinguished road

      0  

    Default


    Awesome thanks!

Thread Participants: 1