STCOM-1224 Include pagination height in MCL container height calculat…

…ion (#2165) * STCOM-1224 Include pagination height in MCL container height calculation * STCOM-1224 Ignore paginationHeight for pagingTypes other than PREV_NEXT
folio-org · Nov 6, 2023 · e88756b · e88756b
1 parent 95268dd
commit e88756b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,7 @@
 * Make `<SearchField>` support input and textarea as an input field. Refs STCOM-1220.
 * Add support for new match option `containsAll` in `<AdvancedSearch>`. Refs STCOM-1223.
 * Ensure CSS visibility of datepicker's year input number spinner. Refs STCOM-1225.
+* Include pagination height in MCL container height calculation. Refs STCOM-1224.
 
 ## [12.0.0](https://github.com/folio-org/stripes-components/tree/v12.0.0) (2023-10-11)
 [Full Changelog](https://github.com/folio-org/stripes-components/compare/v11.0.0...v12.0.0)

diff --git a/lib/MultiColumnList/MCLRenderer.js b/lib/MultiColumnList/MCLRenderer.js
@@ -1749,9 +1749,11 @@ class MCLRenderer extends React.Component {
     if (height !== undefined) {
       // if we have a totalCount, we can set size based on that... if not, the receivedRows count.
       const scrollBarHeight = 20;
+      const paginationHeight = pagingType === pagingTypes.PREV_NEXT ? this.paginationHeight : 0;
+
       newHeight = Math.max(
         ((pagingType !== pagingTypes.SCROLL ? receivedRows : totalCount || receivedRows) * minimumRowHeight),
-        (height - this.headerHeight - scrollBarHeight)
+        (height - this.headerHeight - scrollBarHeight - paginationHeight)
       );
 
       const rowContainerStyles = {