Success! Looks like we've fixed this one. According to our records the fix was applied for EXTJS-4875 in a recent build.
  1. #1
    Sencha Premium Member
    Join Date
    May 2011
    Posts
    53
    Vote Rating
    -1
    mrinsan is infamous around these parts mrinsan is infamous around these parts mrinsan is infamous around these parts

      0  

    Default 4.1 beta - Locking grid do not respond to cellclick event

    Since my earlier bug report was CLOSED, so am starting a new thread, with little more info on it

    cellclick event in locking grid do not respond.

    I think it is a similar issue as
    EXTJSIV-3775 : 'itemclick' event doesn't bubble when using locking grid

    cellclick event is not documented, this raises a doubt if it will be supported in future versions?


    Test case
    Code:
    Ext.onReady(function() {
        Ext.QuickTips.init();
    
    
        // sample static data for the store
        var myData = [
            ['3m Co',                      71.72, 0.02,  0.03,  '9/1 12:00am'],
            ['Alcoa Inc',                  29.01, 0.42,  1.47,  '9/1 12:00am'],
            ['Altria Group Inc',           83.81, 0.28,  0.34,  '9/1 12:00am'],
            ['American Express Company',   52.55, 0.01,  0.02,  '9/1 12:00am'],
            ['American Group, Inc.',       64.13, 0.31,  0.49,  '9/1 12:00am'],
        ];
    
    
        var store = Ext.create('Ext.data.ArrayStore', {
            fields: [
               {name: 'company'},
               {name: 'price',      type: 'float'},
               {name: 'change',     type: 'float'},
               {name: 'pctChange',  type: 'float'},
               {name: 'lastChange', type: 'date', dateFormat: 'n/j h:ia'}
            ],
            data: myData
        });
    
    
        // create the Grid
        var grid = Ext.create('Ext.grid.Panel', {
            store      : store,
            selType    :'cellmodel',
            columns: [{text:'Company', width:200, dataIndex:'company',locked : true },
                      {text:'Price'  , width:125, dataIndex:'price'    },
                      {text:'Change' , width:125, dataIndex:'change'   },
                      {text:'%Change', width:125, dataIndex:'pctChange'},
                      {text:'Updated', width:35,  dataIndex:'lastChange'}],
            height: 350,
            width : 600,
            title : 'Locking Grid Column',
            renderTo:'grid-example',
        });
    });
    
    grid.on(  'cellclick',
              function(){
                 console.log(arguments);
              },
              this
           )

  2. #2
    Sencha - Senior Software Engineer mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    38,215
    Vote Rating
    1044
    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

    Even though cellclick is not documented, itemclick doesn't do everything it should... it doesn't give you the column index.
    Mitchell Simoens @SenchaMitch
    Sencha Inc, Senior Software Engineer
    ________________
    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 Premium Member
    Join Date
    May 2011
    Posts
    53
    Vote Rating
    -1
    mrinsan is infamous around these parts mrinsan is infamous around these parts mrinsan is infamous around these parts

      0  

    Default Thanks for assigning it a ticket

    That being said, i will continue to use and depend on cellclick event (for now without a locking grid)

  4. #4
    Sencha Premium Member
    Join Date
    May 2011
    Posts
    53
    Vote Rating
    -1
    mrinsan is infamous around these parts mrinsan is infamous around these parts mrinsan is infamous around these parts

      0  

    Default seems like this issue is still there in 4.1.1 RC2

    can you confirm if this will be solved in this release ?

  5. #5
    Sencha Premium Member
    Join Date
    Dec 2011
    Posts
    11
    Vote Rating
    0
    zmathew is on a distinguished road

      0  

    Default EXTJSIV-4875 - Status Check

    Will this be resolved in 4.2 ? It does not seem to be resolved in 4.1.3 or 4.2 beta

  6. #6
    Sencha Premium Member
    Join Date
    Apr 2009
    Posts
    236
    Vote Rating
    15
    jhoweaa will become famous soon enough

      0  

    Default

    An update on the status of this bug would be greatly appreciated. We need to use a locking column in our application and we need the cellclick event as well.

    Thanks!

  7. #7
    Sencha Premium Member
    Join Date
    Dec 2011
    Posts
    11
    Vote Rating
    0
    zmathew is on a distinguished road

      0  

    Default

    jhoweaa,


    I did come across the following override. I am not sure if it will fix it. But you can try it out and check if there you have no other alternatives.


    Ext.define( 'EXTJSIV-4875.grid.Panel', {
    extend: 'Ext.grid.Panel',
    bothCfgCopy: [ 'invalidateScrollerOnRefresh',
    'hideHeaders',
    'enableColumnHide',
    'enableColumnMove',
    'enableColumnResize',
    'sortableColumns',
    'listeners' // added this so that ExtJs is registering the listeners if injected by config
    ],

    normalCfgCopy: [ 'verticalScroller',
    'verticalScrollDock',
    'verticalScrollerType',
    'scroll'
    ],

    lockedCfgCopy: [ ],

    /// overridden to delegate events to locked/normal grid depending on the config above
    on: function( eventName, fn, scope, options ){
    // if lockedColumn feature is used, delegate listeners to normalGrid/lockedGrid
    if( this.lockable ){
    if( this.normalGrid && this._issetInCfgCopy( 'listeners', this.normalCfgCopy ) ){
    this.normalGrid.on( eventName, fn, scope, options );
    }

    if( this.lockedGrid && this._issetInCfgCopy( 'listeners', this.lockedCfgCopy ) ){
    this.lockedGrid.on( eventName, fn, scope, options );
    }

    return;
    }

    this.callParent( arguments );
    },

    /**
    * Returns true, if a config section is copied to special grid (locked or normal).
    *
    * @param {string} configSection The name of the config section.
    * @param {Array} specialConfig The special config: this.normalCfgCopy or this.lockedCfgCopy
    * @return {Boolean}
    */
    _issetInCfgCopy: function( configSection, specialConfig ){
    var config = Ext.Array.merge( this.bothCfgCopy, specialConfig );
    var i, ln;

    for( i = 0, ln = config.length; i < ln; i++ ){
    if( config[i] === configSection ){
    return true;
    }
    }

    return false;
    }

    });

  8. #8
    Sencha Premium Member
    Join Date
    Dec 2011
    Posts
    11
    Vote Rating
    0
    zmathew is on a distinguished road

      0  

  9. #9
    Sencha - Ext JS Dev Team Animal's Avatar
    Join Date
    Mar 2007
    Location
    Notts/Redwood City
    Posts
    30,555
    Vote Rating
    60
    Animal will become famous soon enough Animal will become famous soon enough

      0  

    Default

    Yes, this is fixed.