Issue 1241 #1580

Merged
merged 3 commits into from Apr 3, 2013

Projects

None yet

4 participants

Contributor

Improves Pull Request #1302 because $.attr('checked') does not work for reading checked state of a checkbox.
I think this feature is necessary because for large models (de)selecting all fields needs lots of clicks, even when using the existing feature (clicking on the model/relation name).

Coverage decreased (-0.04%) when pulling 1071a0a on DarthMax:issue_1241_fix into 8533e5a on sferik:master.

View Details

Collaborator

Can you please put the JS behavior in the app/assets/javascripts/rails_admin/ui.coffee file ?

Contributor
DarthMax commented Apr 2, 2013

Okay I have moved the stuff to ui.coffee ;)

Coverage decreased (-0.0%) when pulling 85fe0a4 on DarthMax:issue_1241_fix into 8533e5a on sferik:master.

View Details

Collaborator

Nice, thanks!

@bbenezech bbenezech merged commit bb64c94 into sferik:master Apr 3, 2013

1 check failed

default The Travis build could not complete due to an error
Details

Hi DarthMax , Thanks for the feature!
I would like to make this select/unselect checkbox work for each field(including the associated fields), could you give me a hint to do this?
I tried but failed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment