Thank you for reporting this bug. We will make it our priority to review this report.
  1. 16 Apr 2009 7:42 AM #1
    robw
    robw is offline
    Ext User
    Join Date
    Oct 2007
    Posts
    83
    Vote Rating
    0
    robw is on a distinguished road
      0  

    Exclamation [FIXED][3.0] Ext.Select change in behaviour?

    [FIXED][3.0] Ext.Select change in behaviour?


    Hi

    I have been using the following code for ages. Until recently it has worked but now is broken (i am using latest code from SVN).

    PHP Code:
    Ext.select('p', true, 'container').addClassOnOver('hi'); 
    Should add the class to all the <p> tags. but now hovering over any of the <p> tags adds the class to the last <p>.

    Any ideas what's going on?!

    Thanks
    Rob

    PHP Code:
    <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
    <    html xmlns="http://www.w3.org/1999/xhtml" >
    <    head>
      <    title></title>
      <    script type="text/javascript" src="/ext/adapter/ext/ext-base.js"></script>
        <script type="text/javascript" src="/ext/ext-all-debug.js"></script>
      <style type="text/css">.hi { background-color: red; }</style>
    </head>
    <body>

    <div id="container">
        <p>1</p>
        <p>2</p>
        <p>3</p>
        <p>4</p>
    </div>

    <script type="text/javascript">
        Ext.onReady(function() {
            Ext.select('p', true, 'container').addClassOnOver('hi'); 
        });
    </script>
    </body>
    </html> 
    Last edited by robw; 17 Apr 2009 at 2:00 AM. Reason: added 2nd param to select(). doesn't affect bug but rules out flyweight causing problem.
    Reply With Quote Reply With Quote
  2. 16 Apr 2009 5:45 PM #2
    jsakalos
    jsakalos is online now
    Sencha - Support Team jsakalos's Avatar
    Join Date
    Apr 2007
    Location
    Slovakia
    Posts
    26,130
    Vote Rating
    81
    jsakalos is a splendid one to behold jsakalos is a splendid one to behold jsakalos is a splendid one to behold jsakalos is a splendid one to behold jsakalos is a splendid one to behold jsakalos is a splendid one to behold jsakalos is a splendid one to behold
      0  
    Send a message via Skype™ to jsakalos

    Default


    For me, (svn 3726) only 4th paragraph highlights on over. Do you want me to move this thread to bugs?
    Jozef Sakalos, aka Saki

    A lot of valuable info at:
    Saki's Extensions and Plugins
    Saki's Extensions and Plugins Docs
    Saki's Examples, Latest: Grid in Card Layout
    Saki's Blog, Featured: Writing a Big Application in Ext, Latest: Grid MultiSearch Plugin Video
    Reply With Quote Reply With Quote
  3. 17 Apr 2009 1:07 AM #3
    robw
    robw is offline
    Ext User
    Join Date
    Oct 2007
    Posts
    83
    Vote Rating
    0
    robw is on a distinguished road
      0  

    Default


    Definitely, thanks.
    Reply With Quote Reply With Quote
  4. 17 Apr 2009 2:12 AM #4
    Condor
    Condor is offline
    Sencha - Community Support Team Condor's Avatar
    Join Date
    Mar 2007
    Location
    The Netherlands
    Posts
    24,251
    Vote Rating
    40
    Condor is a splendid one to behold Condor is a splendid one to behold Condor is a splendid one to behold Condor is a splendid one to behold Condor is a splendid one to behold Condor is a splendid one to behold Condor is a splendid one to behold
      0  

    Default


    Ext.select() is defined in core\core\CompositeElementLite.js, but it should be redefined in core\CompositeElement.js.

    Simply adding:
    Code:
    Ext.select = Ext.Element.select;
    should fix things.
    Reply With Quote Reply With Quote
  5. 17 Apr 2009 2:37 AM #5
    robw
    robw is offline
    Ext User
    Join Date
    Oct 2007
    Posts
    83
    Vote Rating
    0
    robw is on a distinguished road
      0  

    Default


    Great, that fixes it thanks.

    I'm confused about what should be included in the build though.

    There is no file "core\core\CompositeElement.js"

    src/ext.jsb includes these for ext-all:

    <include name="core\core\CompositeElementLite.js" />
    <include name="core\CompositeElementLite-more.js" />
    <include name="core\CompositeElement.js" />

    What should it be?


    Thanks
    Rob
    Reply With Quote Reply With Quote
  6. 17 Apr 2009 3:20 AM #6
    Condor
    Condor is offline
    Sencha - Community Support Team Condor's Avatar
    Join Date
    Mar 2007
    Location
    The Netherlands
    Posts
    24,251
    Vote Rating
    40
    Condor is a splendid one to behold Condor is a splendid one to behold Condor is a splendid one to behold Condor is a splendid one to behold Condor is a splendid one to behold Condor is a splendid one to behold Condor is a splendid one to behold
      0  

    Default


    My info wasn't 100% correct. I've updated my previous post with the correct info.
    Reply With Quote Reply With Quote
  7. 17 Apr 2009 4:05 AM #7
    robw
    robw is offline
    Ext User
    Join Date
    Oct 2007
    Posts
    83
    Vote Rating
    0
    robw is on a distinguished road
      0  

    Default


    ok, will it also be fixed in the trunk?

    Thanks for your help
    Rob
    Reply With Quote Reply With Quote
  8. 17 Apr 2009 2:11 PM #8
    evant
    evant is online now
    Sencha - Ext JS Dev Team evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    15,082
    Vote Rating
    97
    evant is a splendid one to behold evant is a splendid one to behold evant is a splendid one to behold evant is a splendid one to behold evant is a splendid one to behold evant is a splendid one to behold evant is a splendid one to behold
      0  

    Default


    Added to the trunk, thanks.
    Evan Trimboli
    Sencha Developer
    Twitter - @evantrimboli
    Don't be afraid of the source code!
    Reply With Quote Reply With Quote
  9. 18 Apr 2009 5:09 AM #9
    robw
    robw is offline
    Ext User
    Join Date
    Oct 2007
    Posts
    83
    Vote Rating
    0
    robw is on a distinguished road
      0  

    Default


    briliant, thanks both for the quick action
    Reply With Quote Reply With Quote