1. #1
    Ext JS Premium Member watrboy00's Avatar
    Join Date
    Aug 2007
    Location
    Minneapolis, MN
    Posts
    473
    Vote Rating
    0
    watrboy00 is on a distinguished road

      0  

    Default [2.1][FIXED] Ext.ux.Portal does not properly destroy Ext.ux.Portal.DropZone

    [2.1][FIXED] Ext.ux.Portal does not properly destroy Ext.ux.Portal.DropZone


    a. Ext 2.1
    b. ext-base.js
    c. Not OS Specific
    d. IE6, Works fine in FF2

    e. Simple setup... Tab panel with 2 items.

    Ext.TabPanel
    Items:
    Ext.tree.TreePanel (containerScroll: true, activeTab)
    Ext.ux.Portal

    What steps will reproduce the problem?
    Show Portal
    Show TreePanel
    Show Portal
    Close Portal
    Show TreePanel (should show by default)
    Attempt to drag a TreeNode

    You will get an unspecified error in IE.

    Can you provide a fix for this problem?
    Problem arises because when the Portal is destroyed, it does not properly destroy the DropZone.

    I have modified the example to include the panel's beforeDestroy method and the DropZone unreg method.
    Attached Files
    Eric Berens
    Intranet Systems Manager @ Best Buy
    You should follow me on twitter here.

  2. #2
    Sencha User
    Join Date
    Apr 2012
    Location
    Austin, Texas
    Posts
    4
    Vote Rating
    0
    brian.moeskau is an unknown quantity at this point

      0  

    Default


    Looks good to me. Updated in SVN, thanks for the patch.

  3. #3
    Ext JS Premium Member watrboy00's Avatar
    Join Date
    Aug 2007
    Location
    Minneapolis, MN
    Posts
    473
    Vote Rating
    0
    watrboy00 is on a distinguished road

      0  

    Default


    Thanks Brian.
    Eric Berens
    Intranet Systems Manager @ Best Buy
    You should follow me on twitter here.

  4. #4
    Sencha Premium Member Mjollnir26's Avatar
    Join Date
    Oct 2008
    Location
    Germany
    Posts
    152
    Vote Rating
    0
    Mjollnir26 is on a distinguished road

      0  

    Exclamation Sorry to resurrect this... but...

    Sorry to resurrect this... but...


    I was pulling out my hair, searching for a fix to the "Unspecified Error" we were getting when changing content in the Center-Region of our App.

    The "StartPage" of the App is a Portal in the Center-Region which is then removed and replaced by a GridPanel in the center region.
    The unspecified error popped up on resizing columns in this grid.

    Diff'ing the Portal.txt against the Portal.js from SDK 2.2.1 i found out that the Line

    Code:
    Ext.dd.ScrollManager.unregister( this.portal.body );
    in the "unreg"-Function at the File's Bottom

    is commented out in the current 2.2.1 SDK but has no comment in watrboys "Portal.txt"
    and removeing the Comment made the column resize work again!

    It seems to work fine so far and all, but i'd really appreciate if someone could clarify what the reason was to comment out that line in the 2.2.1 SDK.. I guess there was a good reason to do so?

  5. #5
    Sencha Premium Member Mjollnir26's Avatar
    Join Date
    Oct 2008
    Location
    Germany
    Posts
    152
    Vote Rating
    0
    Mjollnir26 is on a distinguished road

      0  

    Cool


    Found a drawback in uncommenting the line... when reloading the Page with the Portal on it it says "Object expected" and complains about some Flyweights not being there... fixed it for now by surrounding the line with an try-catch but that aint a beautiful solution either....

  6. #6
    Ext User
    Join Date
    Jul 2007
    Location
    Florida
    Posts
    9,996
    Vote Rating
    6
    mjlecomte will become famous soon enough mjlecomte will become famous soon enough

      0  

    Default


    If you're confident you have a new or bug that remains suggest you post a new thread. You haven't followed the directions when posting a new thread in bugs though, that is post working code that reproduces a problem. No one can fix a problem they can't reproduce themselves.

  7. #7
    Sencha Premium Member Mjollnir26's Avatar
    Join Date
    Oct 2008
    Location
    Germany
    Posts
    152
    Vote Rating
    0
    Mjollnir26 is on a distinguished road

      0  

    Default


    Ok then, ill throw together a minimal sample and post that as a new Bug.
    I just thought it would be better to answer in a thread which describes almost the same problem.

  8. #8
    Ext User
    Join Date
    Jul 2007
    Location
    Florida
    Posts
    9,996
    Vote Rating
    6
    mjlecomte will become famous soon enough mjlecomte will become famous soon enough

      0  

    Default


    As you've noticed the problem is the thread is taggged Fixed. Also you indicate "almost" same problem.
    My opinion...

  9. #9
    Sencha Premium Member Mjollnir26's Avatar
    Join Date
    Oct 2008
    Location
    Germany
    Posts
    152
    Vote Rating
    0
    Mjollnir26 is on a distinguished road

      0  

    Default


    It's definitley in the same direction and the fix from watrboy partially corrects it... So i said "almost".

    Btw i just got the sample ready and i am 100% sure now that this is rather a Problem in EXT than with my code.

    Will post sample to Bugs now

  10. #10
    Sencha Premium Member Mjollnir26's Avatar
    Join Date
    Oct 2008
    Location
    Germany
    Posts
    152
    Vote Rating
    0
    Mjollnir26 is on a distinguished road

      0  

    Default


    Followup-Thread in Bugs-Forum:
    http://extjs.com/forum/showthread.php?t=65734