Change the way TablixMembers are searched and used for TablixRowHierarchy so we process the deepest children, taking into account grouping and sorting when we come across them. Probably need to look at columns too.

Fix bug in SumParser to trim dataset name and use correct field. UI fixes.

Author: techyian

assets/reportviewer.net.css

@@ -16,15 +16,20 @@
     overflow: hidden;
 }
 
-.reportparam-date, .reportparam-string, .reportparam-boolean {
-    display: inline-flex;
-    max-height: 35px;
+.reportparam-date, .reportparam-string, .reportparam-boolean {        
     margin-left: 0.5rem;
 }
 
+.reportparam-boolean {
+    display: flex;
+}
+
+.reportparam-boolean .custom-checkbox {
+    align-content: end;
+}
+
 .reportparam-date input, .reportparam-string input {
-    width: auto;
-    margin-left: 0.5rem;
+    width: auto;    
 }
 
 .reportparam-list-select {
@@ -49,6 +54,19 @@
     height: 150px;
 }
 
+.reportparam-list-dropdown.open {
+    position: absolute;
+    border-color: darkgray;
+    border-width: 1px;
+    border-style: solid;
+    background-color: #ffffff;
+    z-index: 1;
+}
+
+.reportparam-list-dropdown.open .custom-checkbox {
+    margin-left: 0.25rem;
+}
+
 .reportparam-list-selectall {
     background-color: transparent;
     border: none;

assets/reportviewer.net.js

@@ -108,6 +108,10 @@
             $('.report-viewer .reportparameters-container').on("click", function (e) {                
                 let load = false;
 
+                if (e.target !== e.currentTarget) {
+                    return;
+                }
+
                 $.each(paramLists, function (idx, list) {                    
                     let dropdownContainers = $(list).find('.reportparam-list-dropdown[class*="open"]');
 
@@ -119,8 +123,8 @@
                         }                        
                     });  
                 });
-                                
-                if (load) {
+
+                if (load || paramLists.length === 0) {
                     self.postReportParameters(); 
                 }
             });

src/ReportViewer.NET.Web/wwwroot/css/site.css

@@ -16,15 +16,20 @@
     overflow: hidden;
 }
 
-.reportparam-date, .reportparam-string, .reportparam-boolean {
-    display: inline-flex;
-    max-height: 35px;
+.reportparam-date, .reportparam-string, .reportparam-boolean {        
     margin-left: 0.5rem;
 }
 
+.reportparam-boolean {
+    display: flex;
+}
+
+.reportparam-boolean .custom-checkbox {
+    align-content: end;
+}
+
 .reportparam-date input, .reportparam-string input {
-    width: auto;
-    margin-left: 0.5rem;
+    width: auto;    
 }
 
 .reportparam-list-select {
@@ -49,6 +54,19 @@
     height: 150px;
 }
 
+.reportparam-list-dropdown.open {
+    position: absolute;
+    border-color: darkgray;
+    border-width: 1px;
+    border-style: solid;
+    background-color: #ffffff;
+    z-index: 1;
+}
+
+.reportparam-list-dropdown.open .custom-checkbox {
+    margin-left: 0.25rem;
+}
+
 .reportparam-list-selectall {
     background-color: transparent;
     border: none;

src/ReportViewer.NET.Web/wwwroot/js/site.js

@@ -108,6 +108,10 @@
             $('.report-viewer .reportparameters-container').on("click", function (e) {                
                 let load = false;
 
+                if (e.target !== e.currentTarget) {
+                    return;
+                }
+
                 $.each(paramLists, function (idx, list) {                    
                     let dropdownContainers = $(list).find('.reportparam-list-dropdown[class*="open"]');
 
@@ -119,8 +123,8 @@
                         }                        
                     });  
                 });
-                                
-                if (load) {
+
+                if (load || paramLists.length === 0) {
                     self.postReportParameters(); 
                 }
             });