(ofbiz-framework) branch release22.01 updated: Fixed: Broken draggable/resizable modals in Helveticus (OFBIZ-12992)

[mleila]

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

mleila pushed a commit to branch release22.01
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/release22.01 by this push:
     new d5fc4ff71b Fixed: Broken draggable/resizable modals in Helveticus 
(OFBIZ-12992)
d5fc4ff71b is described below

commit d5fc4ff71b3eeef8a2c5ac93bb1e3d9db3d363f7
Author: Florian Motteau <florian.mott...@nereide.fr>
AuthorDate: Fri Apr 5 10:17:36 2024 +0200

    Fixed: Broken draggable/resizable modals in Helveticus (OFBIZ-12992)
    
    CSS rules were preventing jQuery UI Dialogs to be draggable
    (!important in CSS overriding top/left inline styles).
    Removes "transition: .5s" on dialog elements : it causes
    slowdowns on drag/resize (a CSS animation is triggered each
    time the user moves/resizes the dialog)
    
    Thanks: Gavinju for reporting and Florian Motteau for providing the PR.
---
 .../helveticus/webapp/helveticus/helveticus-main-theme.less  | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/themes/helveticus/webapp/helveticus/helveticus-main-theme.less 
b/themes/helveticus/webapp/helveticus/helveticus-main-theme.less
index 0ff04c666e..460f68037d 100644
--- a/themes/helveticus/webapp/helveticus/helveticus-main-theme.less
+++ b/themes/helveticus/webapp/helveticus/helveticus-main-theme.less
@@ -97,7 +97,10 @@ span.label{
 }
 
 /*** BUTTON /LINK / NAV STYLES ***/
-a, a:hover, * {
+// * { transition: .5s } is very (too) broad, it causes slowdowns on 
draggable/resizable elements like modals
+// (leads to animations for each new size/position, eventually each pixel...)
+// We should invert the logic here : transitions on chosen elements instead of 
everywhere except some elements.
+a, a:hover, *:not(.ui-dialog, .ui-dialog-content) {
   text-decoration:none;
   -o-transition:.5s;
   -ms-transition:.5s;
@@ -1391,8 +1394,6 @@ form .basic-table,
   }
 }
 .ui-dialog{
-  left: ~"calc(50% - 320px)" !important;
-  top: 2rem !important;
   box-shadow: 0 0 15px 15px @shadow-color !important;
 }
 
@@ -1496,11 +1497,6 @@ form .basic-table,
     }
   }
 }
-.ui-dialog{
-  left: ~"calc(50% - 320px)" !important;
-  top: 2rem !important;
-  box-shadow: 0 0 15px 15px @shadow-color !important;
-}
 
 .hidden{
   display: none;