[Libreoffice-commits] online.git: Branch 'distro/collabora/co-4-2' - loleaflet/src

[Pranam Lashkari (via logerrit)]

 loleaflet/src/control/Control.JSDialogBuilder.js |   21 +++++++++++++++++++--
 1 file changed, 19 insertions(+), 2 deletions(-)

New commits:
commit 5995355daaaa2e7163eaa569a42590580358f70d
Author:     Pranam Lashkari <lpranam at collabora.com>
AuthorDate: Mon May 18 20:44:49 2020 +0530
Commit:     Andras Timar <andras.timar at collabora.com>
CommitDate: Wed Jun 10 16:44:29 2020 +0200

    leaflet: Strange behavior of Paragraph Spacing controls
    
    Change Paragraph Spacing/indent on the mobile wizard with uno buttons,
    paragraph Spacing spinners aren't updated
    
    Change-Id: I0a692790e406a078c5540fe538c15d18cae20af0
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/94433
    Tested-by: Jenkins
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice at gmail.com>
    Reviewed-by: Andras Timar <andras.timar at collabora.com>
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/96011

diff --git a/loleaflet/src/control/Control.JSDialogBuilder.js b/loleaflet/src/control/Control.JSDialogBuilder.js
index 8c8426c9e..7045a77d3 100644
--- a/loleaflet/src/control/Control.JSDialogBuilder.js
+++ b/loleaflet/src/control/Control.JSDialogBuilder.js
@@ -118,9 +118,9 @@ L.Control.JSDialogBuilder = L.Control.extend({
 			// It listens server state changes using GetControlState
 			// to avoid unit conversion
 			builder.map.on('commandstatechanged', function(e) {
-				var value = e.state[data.id];
+				var value = e.state[builder._getFieldFromId(data.id)];
 				if (value) {
-					value = parseFloat(value);
+					value = parseFloat(value.replace(',', '.'));
 					$(controls.spinfield).attr('value', value);
 				}
 			}, this);
@@ -1164,6 +1164,23 @@ L.Control.JSDialogBuilder = L.Control.extend({
 		return null;
 	},
 
+	_getFieldFromId: function(id) {
+		switch (id) {
+		case 'aboveparaspacing':
+			return 'upper';
+		case 'belowparaspacing':
+			return 'lower';
+		case 'beforetextindent':
+			return 'left';
+		case 'aftertextindent':
+			return 'right';
+		case 'firstlineindent':
+			return 'firstline';
+		default:
+			return id;
+		}
+	},
+
 	_getTitleForControlWithId: function(id) {
 		switch (id) {