This is an automated email from the ASF dual-hosted git repository.

svenmeier pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/wicket.git


The following commit(s) were added to refs/heads/master by this push:
     new edaa76c  WICKET-6613 don't use null
edaa76c is described below

commit edaa76c9bcad19bf04b8a90a419f437be75ab2cd
Author: Sven Meier <[email protected]>
AuthorDate: Thu Aug 29 20:43:26 2019 +0200

    WICKET-6613 don't use null
    
    as it removes the key and the default height setting will overwrite
---
 .../apache/wicket/extensions/ajax/markup/html/modal/ModalWindow.java   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/modal/ModalWindow.java
 
b/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/modal/ModalWindow.java
index 808a9e8..7cc3f36 100644
--- 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/modal/ModalWindow.java
+++ 
b/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/modal/ModalWindow.java
@@ -1023,7 +1023,8 @@ public class ModalWindow extends Panel
                }
                else
                {
-                       settings.put("height", (Object)null);
+                       // WICKET-6613 null would remove the key, so use false 
instead 
+                       settings.put("height", false);
                }
 
                settings.put("resizable", isResizable());

Reply via email to