Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

3440_ux_fix_select_add3 #3694

Closed
wants to merge 1 commit into from
Closed

Conversation

Kixo
Copy link
Contributor

@Kixo Kixo commented May 31, 2014

No description provided.

@Kixo
Copy link
Contributor Author

Kixo commented May 31, 2014

@bnisevic
Copy link

Works! Good for merge!

TESTING INSTRUCTIONS:
Go to a items list, for example index.php?option=com_content&view=articles and select checkboxes one by one in that way that all are selected. When all of them are selected then "Check All" checkbox should be selected. If you deselect only one from the list "Check All" checkbox should become deselected.
(Refresh your page in order to avoid cached code.)

@b2z
Copy link
Member

b2z commented May 31, 2014

@test passed using this instructions on Chrome:

Go to a items list, for example index.php?option=com_content&view=articles and select checkboxes one by one in that way that all are selected. When all of them are selected then "Check All" checkbox should be selected. If you deselect only one from the list "Check All" checkbox should become deselected

@Bakual Bakual closed this in 17721f8 May 31, 2014
@Bakual
Copy link
Contributor

Bakual commented May 31, 2014

Thanks for this improvement and the tests!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants