PDA

View Full Version : [FIXED] Ext.selection.CheckboxModel misses some critical events



gatherworks
28 Mar 2012, 12:08 AM
Need to listen for these events specifically:


deselect: function(model, record, index)

select: function(model, record, index)



However, they are not options when trying to add event handlers.

Also, there is no option to set showHeaderCheckbox. When I add it as a custom property and set it to false, the value is converted to a string 'false' and therefor does not work. It would be nice to have a way to add custom non-string properties.

aconran
28 Mar 2012, 9:00 AM
It would be nice to have a way to add custom non-string properties.

Custom configurations are added as strings by default. If you provide them a type they will be added as non-string's. For example

showHeaderCheckbox: false


As to the events, we will make sure we address these.

gatherworks
28 Mar 2012, 9:24 AM
Thanks for the info on getting the add by type! However, it does not completely work as expected :)

I can add the property with showHeaderCheckbox: false and I see it in the config, but not in the generated JSON code. Now if I check the box, I see it in the generated code as showHeaderCheckbox: true.

I also tried to add it as showHeaderCheckbox: true and then uncheck the box. No luck. It only affects the JSON code when set to true, which is the default value (and not what I need).

aconran
28 Mar 2012, 9:35 AM
You are correct. It appears that custom added configurations which are boolean do not go into the codegen when they are set to false.

Thanks for the report.

aconran
30 Mar 2012, 10:21 AM
Also, there is no option to set showHeaderCheckbox.

There is no configuration that I am aware of called showHeaderCheckbox. The events bug has been resolved and will be in the next build > 341. The boolean custom property bug has also been addressed.

gatherworks
30 Mar 2012, 10:23 AM
Thanks! When do you anticipate the next build to go out?

showHeaderCheckbox was added in ExtJs 4.1... I am using ExtJs 4.1 includes when using the generated project from Designer.

aconran
30 Mar 2012, 2:42 PM
This evenings build will be the candidate for the next one to go out... so should be early next week.

I searched for showHeaderCheckbox in 4.1 RC1 and didn't see it. When we do the 4.1 target framework we'll do a full sweep of the new configurations, events, etc

gatherworks
30 Mar 2012, 3:54 PM
Thanks! It is definitely there and works as expected.

http://localhost/extjs41/docs/#!/api/Ext.selection.CheckboxModel (local copy of docs)

showHeaderCheckbox (http://localhost/extjs41/docs/#!/api/Ext.selection.CheckboxModel-cfg-showHeaderCheckbox) : Boolean (http://localhost/extjs41/docs/#!/api/Boolean)
False to not display the header checkbox at the top of the column.
Defaults to: true