diff options
author | Elliot Smith <elliot.smith@intel.com> | 2016-01-15 13:00:51 +0200 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-01-15 16:29:59 +0000 |
commit | 1a4b203ca59bcf953b030dc582e298ce13cf3470 (patch) | |
tree | 6db26e71ca5b382853560e58bfbd4275fe68026d | |
parent | 809046c6fbd544907b5d5f3bb554b71724c74661 (diff) | |
download | poky-1a4b203ca59bcf953b030dc582e298ce13cf3470.tar.gz |
bitbake: toastergui: switch off filter highlights when inactive
In ToasterTables with multiple columns which allow filtering
(e.g. all builds), selecting one filter, then a second filter
(e.g. selecting "failed builds" then "outcome" for all builds),
would result in both filters being highlighted at the same time.
Fix this by removing the "active" highlight on all column filter
buttons when a new filter value is submitted (via the filter modal).
NB to enable this, added a data-filter-on attribute to all
column filter buttons to make them easy to select.
[YOCTO #8738]
(Bitbake rev: 7347ad0d4baace593751b44a86ab8e11a04a02b6)
Signed-off-by: Elliot Smith <elliot.smith@intel.com>
Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | bitbake/lib/toaster/toastergui/static/js/table.js | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/bitbake/lib/toaster/toastergui/static/js/table.js b/bitbake/lib/toaster/toastergui/static/js/table.js index fa01ddf47e..63f8a1fed7 100644 --- a/bitbake/lib/toaster/toastergui/static/js/table.js +++ b/bitbake/lib/toaster/toastergui/static/js/table.js | |||
@@ -248,7 +248,7 @@ function tableInit(ctx){ | |||
248 | 248 | ||
249 | /* Setup the filter button */ | 249 | /* Setup the filter button */ |
250 | if (col.filter_name){ | 250 | if (col.filter_name){ |
251 | var filterBtn = $('<a href="#" role="button" class="pull-right btn btn-mini" data-toggle="modal"><i class="icon-filter filtered"></i></a>'); | 251 | var filterBtn = $('<a href="#" role="button" data-filter-on="' + col.filter_name + '" class="pull-right btn btn-mini" data-toggle="modal"><i class="icon-filter filtered"></i></a>'); |
252 | 252 | ||
253 | filterBtn.data('filter-name', col.filter_name); | 253 | filterBtn.data('filter-name', col.filter_name); |
254 | filterBtn.prop('id', col.filter_name); | 254 | filterBtn.prop('id', col.filter_name); |
@@ -565,6 +565,12 @@ function tableInit(ctx){ | |||
565 | $("#filter-modal-form-"+ctx.tableName).submit(function(e){ | 565 | $("#filter-modal-form-"+ctx.tableName).submit(function(e){ |
566 | e.preventDefault(); | 566 | e.preventDefault(); |
567 | 567 | ||
568 | /* remove active status from all filter buttons so that only one filter | ||
569 | can be active at a time */ | ||
570 | $('[data-filter-on]').each(function (index, filterBtn) { | ||
571 | filterBtnActive($(filterBtn), false); | ||
572 | }); | ||
573 | |||
568 | tableParams.filter = $(this).find("input[type='radio']:checked").val(); | 574 | tableParams.filter = $(this).find("input[type='radio']:checked").val(); |
569 | 575 | ||
570 | var filterBtn = $("#" + tableParams.filter.split(":")[0]); | 576 | var filterBtn = $("#" + tableParams.filter.split(":")[0]); |