[Libreoffice-commits] online.git: Branch 'distro/collabora/collabora-online-2-1' - 5 commits - loleaflet/dist loleaflet/src

Aditya Dewan iit2015097 at iiita.ac.in
Thu Jul 27 09:39:01 UTC 2017


 loleaflet/dist/toolbar/toolbar.js        |   27 ++++++++++++++++++++++-----
 loleaflet/src/control/Control.Menubar.js |   28 ++++++++++++++++++----------
 loleaflet/src/layer/tile/TileLayer.js    |    4 +++-
 3 files changed, 43 insertions(+), 16 deletions(-)

New commits:
commit f73acaa3a6512bc2e4eea18214dbdb8ee489256c
Author: Aditya Dewan <iit2015097 at iiita.ac.in>
Date:   Wed Jun 21 16:23:45 2017 +0530

    fix js lint error
    
    Change-Id: I42183dc8865a6936de26e18459ab985e2f9bba1f
    Reviewed-on: https://gerrit.libreoffice.org/39046
    Reviewed-by: pranavk <pranavk at collabora.co.uk>
    Tested-by: pranavk <pranavk at collabora.co.uk>

diff --git a/loleaflet/dist/toolbar/toolbar.js b/loleaflet/dist/toolbar/toolbar.js
index 2d842761..c0ea64e9 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -748,7 +748,7 @@ function onSearch(e) {
 		toolbar.hide('cancelsearch');
 	}
 	else {
-		if(map.getDocType() === 'text')
+		if (map.getDocType() === 'text')
 			map.search(L.DomUtil.get('search-input').value, false, '', 0, true /* expand search */);
 		toolbar.enable('searchprev');
 		toolbar.enable('searchnext');
commit 361cdf8a916bc1bf01f8696631d3bb35cf7ac086
Author: Aditya Dewan <iit2015097 at iiita.ac.in>
Date:   Sat Jun 17 08:48:13 2017 +0530

    disable search as type from the dialog box for calc/impress
    
    Change-Id: I93fa810876c405ea29521ecd90d4cf70affc7994
    Reviewed-on: https://gerrit.libreoffice.org/38904
    Reviewed-by: pranavk <pranavk at collabora.co.uk>
    Tested-by: pranavk <pranavk at collabora.co.uk>

diff --git a/loleaflet/dist/toolbar/toolbar.js b/loleaflet/dist/toolbar/toolbar.js
index aa5e4cfe..2d842761 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -741,10 +741,19 @@ function selectItem(item, func)
 }
 function onSearch(e) {
 	var toolbar = w2ui['toolbar-down'];
-	map.search(L.DomUtil.get('search-input').value, false, '', 0, true /* expand search */);
-	toolbar.enable('searchprev');
-	toolbar.enable('searchnext');
-	toolbar.show('cancelsearch');
+	// conditionally disabling until, we find a solution for tdf#108577
+	if (L.DomUtil.get('search-input').value === '') {
+		toolbar.disable('searchprev');
+		toolbar.disable('searchnext');
+		toolbar.hide('cancelsearch');
+	}
+	else {
+		if(map.getDocType() === 'text')
+			map.search(L.DomUtil.get('search-input').value, false, '', 0, true /* expand search */);
+		toolbar.enable('searchprev');
+		toolbar.enable('searchnext');
+		toolbar.show('cancelsearch');
+	}
 }
 
 function onSearchKeyPress(e) {
diff --git a/loleaflet/src/control/Control.Menubar.js b/loleaflet/src/control/Control.Menubar.js
index 9fe98e3f..fdd63bcd 100644
--- a/loleaflet/src/control/Control.Menubar.js
+++ b/loleaflet/src/control/Control.Menubar.js
@@ -512,11 +512,13 @@ L.Control.Menubar = L.Control.extend({
 					var xElem = 0, yElem = 0;
 					var maxH = window.innerHeight, maxW = window.innerWidth;
 
-					$('#findthis').on('input', function() {
-						if (this.value.length != 0) {
-							map.search(this.value, false, '', 0, true);
-						}
-					});
+					if (map.getDocType() === 'text') {
+						$('#findthis').on('input', function() {
+							if (this.value.length != 0) {
+								map.search(this.value, false, '', 0, true);
+							}
+						});
+					}
 					$('.vex-content').on('mousedown', function(e) {
 						selected = this;
 						selected.style.cursor = 'move';
commit 822985a5ca4546bd5354b6d4ed50d3bb08e3f68d
Author: Pranav Kant <pranavk at collabora.co.uk>
Date:   Wed Jun 21 12:12:25 2017 +0530

    Disable Copy from calc cells and formula bar
    
    ... of course only when DisableCopy is mentioned.
    
    (cherry picked from commit 6e46c760ce6928e5c2120499a44d79529e263e07)
    (cherry picked from commit 202b9974d74469355203f9e38400c7b6a6a50a7d)
    
    Change-Id: I843b7cab2e3fe2dd691396e4702281a6bc953947

diff --git a/loleaflet/dist/toolbar/toolbar.js b/loleaflet/dist/toolbar/toolbar.js
index 94b17e42..aa5e4cfe 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -938,6 +938,14 @@ map.on('wopiprops', function(e) {
 		w2ui['toolbar-up'].hide('save');
 		w2ui['toolbar-up'].hide('savebreak');
 	}
+	if (e.DisableCopy) {
+		$('input#formulaInput').bind('copy', function(evt) {
+			evt.preventDefault();
+		});
+		$('input#addressInput').bind('copy', function(evt) {
+			evt.preventDefault();
+		});
+	}
 });
 
 map.on('doclayerinit', function () {
@@ -1571,7 +1579,7 @@ function getUserItem(viewId, userName, extraInfo, color) {
 	var html = '<tr class="' + className + '" id="user-' + viewId + '" onclick="onUseritemClicked(event)">' +
 	             '<td class=usercolor style="background-color: ' + color  +';">';
 	if (extraInfo !== undefined && extraInfo.avatar !== undefined) {
-		html += '<img src="' + extraInfo.avatar + '" width="32" height="32" />'
+		html += '<img src="' + extraInfo.avatar + '" width="32" height="32" />';
 	}
 
 	// TODO: Add mail and other links as sub-menu.
diff --git a/loleaflet/src/layer/tile/TileLayer.js b/loleaflet/src/layer/tile/TileLayer.js
index 109569ce..187d26d1 100644
--- a/loleaflet/src/layer/tile/TileLayer.js
+++ b/loleaflet/src/layer/tile/TileLayer.js
@@ -509,7 +509,9 @@ L.TileLayer = L.GridLayer.extend({
 
 	_onCellFormulaMsg: function (textMsg) {
 		var formula = textMsg.substring(13);
-		this._selectionTextContent = formula;
+		if (!this._map['wopi'].DisableCopy) {
+			this._selectionTextContent = formula;
+		}
 		this._map.fire('cellformula', {formula: formula});
 	},
 
commit 5865b31ea5aef28666bf6eff36b333151c133e6d
Author: Jan Holesovsky <kendy at collabora.com>
Date:   Wed Jun 7 17:36:40 2017 +0200

    loleaflet: Position the Find & replace consistently, and in line with desktop.
    
    Change-Id: I5d9d93339d74f5a9cda304fba876c76cb5391408

diff --git a/loleaflet/src/control/Control.Menubar.js b/loleaflet/src/control/Control.Menubar.js
index d29bd46c..9fe98e3f 100644
--- a/loleaflet/src/control/Control.Menubar.js
+++ b/loleaflet/src/control/Control.Menubar.js
@@ -25,7 +25,6 @@ L.Control.Menubar = L.Control.extend({
 				{name: _('Cut'), type: 'unocommand', uno: '.uno:Cut'},
 				{name: _('Copy'), type: 'unocommand', uno: '.uno:Copy'},
 				{name: _('Paste'), type: 'unocommand', uno: '.uno:Paste'},
-				{type: 'separator'},
 				{name: _('Select all'), type: 'unocommand', uno: '.uno:SelectAll'},
 				{type: 'separator'},
 				{name: _('Find & Replace'), id: 'findandreplace', type: 'action'},
@@ -190,10 +189,9 @@ L.Control.Menubar = L.Control.extend({
 				{name: _('Cut'), type: 'unocommand', uno: '.uno:Cut'},
 				{name: _('Copy'), type: 'unocommand', uno: '.uno:Copy'},
 				{name: _('Paste'), type: 'unocommand', uno: '.uno:Paste'},
+				{name: _('Select all'), type: 'unocommand', uno: '.uno:SelectAll'},
 				{type: 'separator'},
-				{name: _('Find & Replace'), id: 'findandreplace', type: 'action'},
-				{type: 'separator'},
-				{name: _('Select all'), type: 'unocommand', uno: '.uno:SelectAll'}]
+				{name: _('Find & Replace'), id: 'findandreplace', type: 'action'}]
 			},
 			{name: _('View'), id: 'view', type: 'menu', menu: [
 				{name: _('Full screen'), id: 'fullscreen', type: 'action'},
@@ -252,10 +250,9 @@ L.Control.Menubar = L.Control.extend({
 				{name: _('Cut'), type: 'unocommand', uno: '.uno:Cut'},
 				{name: _('Copy'), type: 'unocommand', uno: '.uno:Copy'},
 				{name: _('Paste'), type: 'unocommand', uno: '.uno:Paste'},
+				{name: _('Select all'), type: 'unocommand', uno: '.uno:SelectAll'},
 				{type: 'separator'},
-				{name: _('Find & Replace'), id: 'findandreplace', type: 'action'},
-				{type: 'separator'},
-				{name: _('Select all'), type: 'unocommand', uno: '.uno:SelectAll'}]
+				{name: _('Find & Replace'), id: 'findandreplace', type: 'action'}]
 			},
 			{name: _('View'), id: 'view', type: 'menu', menu: [
 				{name: _('Full screen'), id: 'fullscreen', type: 'action'}]
commit 7c9300f61cb18b0e01fb077db4f52ef4332d189b
Author: Aditya Dewan <iit2015097 at iiita.ac.in>
Date:   Mon Jun 5 18:05:04 2017 +0530

    tdf#97020 follow up - Adding find&replace functionality
    
    added option for calc and impress
    draggable operation now smoother
    
    Change-Id: I46210689a1592478827271385da48a854aece766
    Reviewed-on: https://gerrit.libreoffice.org/38405
    Reviewed-by: pranavk <pranavk at collabora.co.uk>
    Tested-by: pranavk <pranavk at collabora.co.uk>
    (cherry picked from commit 3c121dad3765f3de68390d160210075701cde9b7)

diff --git a/loleaflet/src/control/Control.Menubar.js b/loleaflet/src/control/Control.Menubar.js
index 0bf811d3..d29bd46c 100644
--- a/loleaflet/src/control/Control.Menubar.js
+++ b/loleaflet/src/control/Control.Menubar.js
@@ -191,6 +191,8 @@ L.Control.Menubar = L.Control.extend({
 				{name: _('Copy'), type: 'unocommand', uno: '.uno:Copy'},
 				{name: _('Paste'), type: 'unocommand', uno: '.uno:Paste'},
 				{type: 'separator'},
+				{name: _('Find & Replace'), id: 'findandreplace', type: 'action'},
+				{type: 'separator'},
 				{name: _('Select all'), type: 'unocommand', uno: '.uno:SelectAll'}]
 			},
 			{name: _('View'), id: 'view', type: 'menu', menu: [
@@ -251,6 +253,8 @@ L.Control.Menubar = L.Control.extend({
 				{name: _('Copy'), type: 'unocommand', uno: '.uno:Copy'},
 				{name: _('Paste'), type: 'unocommand', uno: '.uno:Paste'},
 				{type: 'separator'},
+				{name: _('Find & Replace'), id: 'findandreplace', type: 'action'},
+				{type: 'separator'},
 				{name: _('Select all'), type: 'unocommand', uno: '.uno:SelectAll'}]
 			},
 			{name: _('View'), id: 'view', type: 'menu', menu: [
@@ -472,6 +476,7 @@ L.Control.Menubar = L.Control.extend({
 				</tr>\
 			</table>\
 			';
+			var mouseMoveFunc;
 			vex.dialog.open({
 				showCloseButton: true,
 				escapeButtonCloses: true,
@@ -525,7 +530,7 @@ L.Control.Menubar = L.Control.extend({
 						selected.style.cursor = 'default';
 						selected = null;
 					});
-					$('.vex').on('mousemove', function(e) {
+					var mouseMoveFunc = function(e) {
 						xPos = e.pageX;
 						yPos = e.pageY;
 						if (selected !== null) {
@@ -538,7 +543,11 @@ L.Control.Menubar = L.Control.extend({
 								selected.style.top = (yPos - yElem) + 'px';
 							}
 						}
-					});
+					};
+					$(document).on('mousemove', mouseMoveFunc);
+				},
+				afterClose: function(e) {
+					$(document).off('mousemove', mouseMoveFunc);
 				},
 				onSubmit: function(event) {
 					var $vexContent = $(this).parent();


More information about the Libreoffice-commits mailing list