There is a YUI menu on the budget planning page used to list columns in the table which can be hidden or shown. This menu should be replaced with a Bootstrap implementation.
Created attachment 15973 [details] [review] Bug 9766 - Remove YUI menu on budget planning page in favor of different column-hiding system The budget planning table previously used a YUI menu as part of a system for hiding and showing table columns. Instead of trying to convert that sytem to Bootstrap I have used code from the batch modification/deletion pages which uses a list of checkboxes to show and hide table columns. This has the advantage of giving the user visual feedback of what columns are hidden and of offering the user an interface which is familiar from other pages. This patch also moves CSS from a batch-operations-specific CSS file into the main CSS file. Batch modification templates have been modified accordingly. To test, view the budget planning page and choose different "plan by" options (by month, by library, by item type). Test the show/hide action on each individual column and using the "show all" and "hide all" checkboxes.
Created attachment 16054 [details] [review] [SIGNED-OFF] Bug 9766 - Remove YUI menu on budget planning page in favor of different column-hiding system The budget planning table previously used a YUI menu as part of a system for hiding and showing table columns. Instead of trying to convert that sytem to Bootstrap I have used code from the batch modification/deletion pages which uses a list of checkboxes to show and hide table columns. This has the advantage of giving the user visual feedback of what columns are hidden and of offering the user an interface which is familiar from other pages. This patch also moves CSS from a batch-operations-specific CSS file into the main CSS file. Batch modification templates have been modified accordingly. To test, view the budget planning page and choose different "plan by" options (by month, by library, by item type). Test the show/hide action on each individual column and using the "show all" and "hide all" checkboxes. Signed-off-by: David Cook <dcook@prosentient.com.au> Works as described. This interface is also much more visually appealing, user-friendly, and informative than it was when it used the YUI menu.
Owen, can you please rebase and switch back to signed off?
Created attachment 17101 [details] [review] Bug 9766 - Remove YUI menu on budget planning page in favor of different column-hiding system The budget planning table previously used a YUI menu as part of a system for hiding and showing table columns. Instead of trying to convert that sytem to Bootstrap I have used code from the batch modification/deletion pages which uses a list of checkboxes to show and hide table columns. This has the advantage of giving the user visual feedback of what columns are hidden and of offering the user an interface which is familiar from other pages. This patch also moves CSS from a batch-operations-specific CSS file into the main CSS file. Batch modification templates have been modified accordingly. To test, view the budget planning page and choose different "plan by" options (by month, by library, by item type). Test the show/hide action on each individual column and using the "show all" and "hide all" checkboxes. Signed-off-by: David Cook <dcook@prosentient.com.au> Works as described. This interface is also much more visually appealing, user-friendly, and informative than it was when it used the YUI menu.
Owen, Did you have a look at ColVis (http://datatables.net/extras/colvis/)? Maybe it could be useful to use it here, what do you think about it?
Created attachment 17306 [details] [review] Bug 9766: [DRAFT] First try with colvis. Add the colvis DT plugin to the aqplan table
Owen, there is a merge conflict in your patch, could you please fix? And maybe give some feedback about the second? I would like to see the YUI menus gone for 3.12.
Created attachment 17646 [details] [review] Bug 9766 - Remove YUI menu on budget planning page in favor of different column-hiding system The budget planning table previously used a YUI menu as part of a system for hiding and showing table columns. Instead of trying to convert that sytem to Bootstrap I have used code from the batch modification/deletion pages which uses a list of checkboxes to show and hide table columns. This has the advantage of giving the user visual feedback of what columns are hidden and of offering the user an interface which is familiar from other pages. This patch also moves CSS from a batch-operations-specific CSS file into the main CSS file. Batch modification templates have been modified accordingly. To test, view the budget planning page and choose different "plan by" options (by month, by library, by item type). Test the show/hide action on each individual column and using the "show all" and "hide all" checkboxes. Signed-off-by: David Cook <dcook@prosentient.com.au> Works as described. This interface is also much more visually appealing, user-friendly, and informative than it was when it used the YUI menu.
(In reply to comment #7) > maybe give some feedback about the second? I haven't had a chance to test Jonathan's patch, and now it doesn't apply. I don't object to the idea of it, although I think it would be nice to keep this and the batch mod interfaces consistent with regard to showing and hiding columns.
Created attachment 17653 [details] [review] Bug 9766: [DRAFT] First try with colvis. Add the colvis DT plugin to the aqplan table
(In reply to comment #9) > I haven't had a chance to test Jonathan's patch, and now it doesn't apply. I > don't object to the idea of it, although I think it would be nice to keep > this and the batch mod interfaces consistent with regard to showing and > hiding columns. Rebased. Owen, I really like to have your opinion on this DT plugin. Not only for this patch (my goal is not to block this one), but more generally. We can imagine a configuration page for all tables in Koha (some users would like to display some columns, others want to hide them...).
QA comment: Works great. This patch gets rid of the last YUI menu in the intranet interface :) Marked as Passed QA.
Created attachment 17655 [details] [review] Bug 9766 - Remove YUI menu on budget planning page in favor of different column-hiding system The budget planning table previously used a YUI menu as part of a system for hiding and showing table columns. Instead of trying to convert that sytem to Bootstrap I have used code from the batch modification/deletion pages which uses a list of checkboxes to show and hide table columns. This has the advantage of giving the user visual feedback of what columns are hidden and of offering the user an interface which is familiar from other pages. This patch also moves CSS from a batch-operations-specific CSS file into the main CSS file. Batch modification templates have been modified accordingly. To test, view the budget planning page and choose different "plan by" options (by month, by library, by item type). Test the show/hide action on each individual column and using the "show all" and "hide all" checkboxes. Signed-off-by: David Cook <dcook@prosentient.com.au> Works as described. This interface is also much more visually appealing, user-friendly, and informative than it was when it used the YUI menu. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
This patch has been pushed to master and 3.12.x.