Success! Looks like we've fixed this one. According to our records the fix was applied for a bug in our system in a recent build.
  1. #1
    Sencha Premium Member
    Join Date
    Aug 2012
    Posts
    59
    Vote Rating
    1
    jkyoutsey is on a distinguished road

      0  

    Exclamation Grid with grouping feature but no grouped field breaks checkcolumn (working example)

    Grid with grouping feature but no grouped field breaks checkcolumn (working example)


    If I create a grid with the grouping feature but none of the columns are currently grouped then the checkcolumn will not render correctly. The underlying data is modified, but the checkbox does not change its rendered state. If it was checked, it stays checked.
    If you group the "a" field in the example below then the checkboxes will work. Ungroup, they are broken again.
    This means that we cannot use any grouping in grids with checkcolumns unless we always force a column to be in the groupField. This is a blocker for us.
    Code:
    <html>
    <head>
       <script type="text/javascript" src="./extjs-4.2/ext-all-debug-w-comments.js"></script>
       <link rel="stylesheet" type="text/css" href="./extjs-4.2/resources/css/ext-all.css">
    <script>
    Ext.onReady(function () {
       var data = {
          "data" : [
             { "a" : "lorem", "selected":true },
             { "a" : "ipsum", "selected":false }
          ]
       };
       
       var store = Ext.create('Ext.data.JsonStore', {
             data : { 
                items : [
                   { "a" : "lorem", "selected":true },
                   { "a" : "ipsum", "selected":false }
                ]},
             fields : ['a', 'selected'],
             proxy : { 
                type : 'memory',
                reader : {
                   type : 'json',
                   root : 'items'
                }
             }
          });
    
       Ext.create('Ext.container.Container', {
          renderTo : Ext.getBody(),
          margin : '10 10 10 10',
          items : [{
                xtype : 'label',
                text : 'Grouping grid without a groupField causes checkcolumn not to work.  Group by any column and the checkcolumn will work again.'
             },{
                xtype : 'grid',
                store : store,
    
                // Either definition of features creates the issue.
    //            features : Ext.create('Ext.grid.feature.Grouping', {
    //               groupHeaderTpl : Ext.create('Ext.XTemplate', '{name}')
    //            }),
    
                features: [{ftype:'grouping'}],
                columns : [{
                      header : 'a',
                      dataIndex : 'a'
                   }, {
                      xtype : 'checkcolumn',
                      header : 'selected',
                      dataIndex : 'selected'
                   }]
             }]
       });
    });
    </script>
    </head>
    <body>
    </body>
    </html>

  2. #2
    Sencha - Support Team slemmon's Avatar
    Join Date
    Mar 2009
    Location
    Boise, ID
    Posts
    4,800
    Vote Rating
    167
    slemmon is a splendid one to behold slemmon is a splendid one to behold slemmon is a splendid one to behold slemmon is a splendid one to behold slemmon is a splendid one to behold slemmon is a splendid one to behold slemmon is a splendid one to behold

      0  

    Default


    It looks like that bug was fixed in 4.2.1.
    There is a JS error thrown when hovering over the rows of a grouped grid with no groupField which has been filed as a bug:
    http://www.sencha.com/forum/showthread.php?264727
    Are you a Sencha products veteran who has wondered what it might be like to work at Sencha? If so, please reach out to our recruiting manager: sheryl@sencha.com

    C
    heck out all of the 2013 SenchaCon presentations here:
    http://www.sencha.com/blog/senchacon...now-available/

  3. #3
    Sencha Premium Member
    Join Date
    Aug 2012
    Posts
    59
    Vote Rating
    1
    jkyoutsey is on a distinguished road

      0  

    Default


    Thanks, slemmon. We can't move right now but we'll be sure to check out the fix in 4.2.1. Thanks!

Thread Participants: 1

Tags for this Thread