ZK-4960: Listbox doesn't keep scrollbar position when applying hflex=…

…"min" on a listheader in mobile
zkoss · Nov 1, 2024 · 59652f2 · 59652f2
1 parent 2ef6cbb
commit 59652f2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/zkdoc/release-note b/zkdoc/release-note
@@ -41,6 +41,7 @@ ZK 10.1.0
   ZK-5810: Client MVVM: XEL Methods in include
   ZK-5813: a side effect of ZK-5476 for BookCRUD2Test and BookCRUDTest
   ZK-5777: sendRedirect() doesn't work because of the encoded &
+  ZK-4960: Listbox doesn't keep scrollbar position when applying hflex="min" on a listheader in mobile
 
 * Upgrade Notes
   + Remove Htmls.encodeJavaScript(), Strings.encodeJavaScript(), Strings.escape() with Strings.ESCAPE_JAVASCRIPT, and replace them with OWASP Java Encoder APIs instead.

diff --git a/zul/src/main/resources/web/js/zul/mesh/HeadWidget.ts b/zul/src/main/resources/web/js/zul/mesh/HeadWidget.ts
@@ -392,7 +392,7 @@ export class HeadWidget extends zul.Widget<HTMLTableRowElement> {
 					wgt._calcSize(); // yes, we need to do it again.
 				}
 				// ZK-2551: need to restore scroll pos after flexs are fixed
-				if (zk(ebody).hasHScroll() && currentLeft != ebody.scrollLeft) {
+				if (ebody.scrollWidth >= ebody.clientWidth && currentLeft != ebody.scrollLeft) {
 					ebody.scrollLeft = currentLeft;
 					if (ehead)
 						ehead.scrollLeft = currentLeft;