Wip: todos and bugs to fix

Author: sukhwinder33445

asset/js/widget/ActionList.js

@@ -90,7 +90,7 @@ define(["../notjQuery"], function ($) {
         onClick(event) {
             let target = event.currentTarget;
 
-            if (target.matches('a') && (! target.matches('.subject') || event.ctrlKey || event.metaKey)) {
+            if (target.matches('a') && (! target.matches('.subject') || event.ctrlKey || event.metaKey)) { //TODO: don't use icinga specific classes
                 return true;
             }
 
@@ -169,6 +169,7 @@ define(["../notjQuery"], function ($) {
             if (! this.isMultiSelectable || ! this.isPrimary) {
                 return;
             }
+            //TODO: remove icinga specific classes, make this function more generic
 
             let activeItemCount = this.getActiveItems().length;
             let footer = this.list.closest('.container').querySelector('.footer');
@@ -405,6 +406,7 @@ define(["../notjQuery"], function ($) {
          * @param pressedKey Pressed key (`ArrowUp` or `ArrowDown`)
          */
         scrollItemIntoView(item, pressedKey) {
+            //TODO: is not working(never did) in Host/Service groups
             item.scrollIntoView({block: "nearest"});
             let directionalNext = this.getDirectionalNext(item, pressedKey);
 
@@ -501,6 +503,8 @@ define(["../notjQuery"], function ($) {
          * @param detailUrl
          */
         load(detailUrl = null) {
+            //TODO: somehow item is deselected when req is canceled and new req is started
+            //isProcessing is not working as expected
             if (! detailUrl) {
                 let activeItems = this.getActiveItems();
                 if (activeItems.length) {