loleaflet/css/mobilewizard.css                   |    8 ++++++-
 loleaflet/src/control/Control.JSDialogBuilder.js |   26 +++++++++++++++--------
 2 files changed, 24 insertions(+), 10 deletions(-)

New commits:
commit 8f05c156563b882f3e843df54dbd46f142b804a3
Author:     Pedro Pinto Silva <pedro.si...@collabora.com>
AuthorDate: Thu Apr 23 14:11:42 2020 +0200
Commit:     Pedro Pinto da Silva <pedro.si...@collabora.com>
CommitDate: Thu Apr 23 14:31:31 2020 +0200

    Mobile: mWizard: disabled input fields: make sure both image and spinfield 
controls do not stay active
    
    And make sure the js listener is only added when the inputfield is enabled 
(fixing useless callbacks and changes on html)
    
    Change-Id: I4f4f2b4430b1c43143f95ab9d9fe075c8e1f49b8
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92763
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com>
    Tested-by: Pedro Pinto da Silva <pedro.si...@collabora.com>
    Reviewed-by: Pedro Pinto da Silva <pedro.si...@collabora.com>

diff --git a/loleaflet/css/mobilewizard.css b/loleaflet/css/mobilewizard.css
index 1a8919e4e..d2b7d73b6 100644
--- a/loleaflet/css/mobilewizard.css
+++ b/loleaflet/css/mobilewizard.css
@@ -540,8 +540,11 @@ input.spinfield:disabled ~ .spinfieldcontrols * {
        background-color: #f9f9f9;
 }
 
-input.spinfield:disabled ~ .spinfieldcontrols *:active {
+input.spinfield:disabled ~ .spinfieldcontrols *:active, 
input.spinfield:disabled ~ .spinfieldcontrols *:hover, input.spinfield:disabled 
~ .spinfieldcontrols *:focus {
        background-color: #f9f9f9 !important;
+       filter: none;
+       font-size: 100%;
+       font-weight: normal;
 }
 
 /*insert table special layout*/
@@ -701,6 +704,9 @@ div#mobile-wizard-content .spinfieldcontainer 
.spinfieldimage {
 div#mobile-wizard-content .spinfieldcontainer:active .spinfieldimage, 
div#mobile-wizard-content .spinfieldcontainer:focus .spinfieldimage, 
div#mobile-wizard-content .spinfieldcontainer:hover .spinfieldimage {
        opacity: 1 !important;
 }
+div#mobile-wizard-content .spinfieldcontainer .spinfieldimage.disabled {
+       opacity: 0.5 !important;
+}
 #buttonbefore, #buttonafter, #buttonoptimal, #buttonparallel, #buttonnone, 
#buttonthrough, #bottom, #top, #standard, #Bold, #Italic, #Underline, 
#Strikeout{
        margin: 0;
        padding: 0;
diff --git a/loleaflet/src/control/Control.JSDialogBuilder.js 
b/loleaflet/src/control/Control.JSDialogBuilder.js
index 323f69bd7..c9a410feb 100644
--- a/loleaflet/src/control/Control.JSDialogBuilder.js
+++ b/loleaflet/src/control/Control.JSDialogBuilder.js
@@ -1044,8 +1044,10 @@ L.Control.JSDialogBuilder = L.Control.extend({
                if (data.max != undefined)
                        $(spinfield).attr('max', data.max);
 
-               if (data.enabled == 'false')
+               if (data.enabled == 'false') {
                        $(spinfield).attr('disabled', 'disabled');
+                       $(image).addClass('disabled');
+               }
 
                if (data.readOnly === true)
                        $(spinfield).attr('readOnly', 'true');
@@ -1081,17 +1083,23 @@ L.Control.JSDialogBuilder = L.Control.extend({
                });
 
                plus.addEventListener('click', function() {
-                       if (customCallback)
-                               customCallback('spinfield', 'plus', div, 
this.value, builder);
-                       else
-                               builder.callback('spinfield', 'plus', div, 
this.value, builder);
+                       var attrdisabled = $(spinfield).attr('disabled');
+                       if (attrdisabled !== 'disabled') {
+                               if (customCallback)
+                                       customCallback('spinfield', 'plus', 
div, this.value, builder);
+                               else
+                                       builder.callback('spinfield', 'plus', 
div, this.value, builder);
+                       }
                });
 
                minus.addEventListener('click', function() {
-                       if (customCallback)
-                               customCallback('spinfield', 'minus', div, 
this.value, builder);
-                       else
-                               builder.callback('spinfield', 'minus', div, 
this.value, builder);
+                       var attrdisabled = $(spinfield).attr('disabled');
+                       if (attrdisabled !== 'disabled') {
+                               if (customCallback)
+                                       customCallback('spinfield', 'minus', 
div, this.value, builder);
+                               else
+                                       builder.callback('spinfield', 'minus', 
div, this.value, builder);
+                       }
                });
 
                if (data.hidden)
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to