[Libreoffice-commits] online.git: loleaflet/src

Jan Holesovsky (via logerrit) logerrit at kemper.freedesktop.org
Wed Nov 13 10:56:38 UTC 2019


 loleaflet/src/control/Control.Toolbar.js |    7 -------
 1 file changed, 7 deletions(-)

New commits:
commit d066c88fea030d173d57e3558e5227cd4cb2ecc7
Author:     Jan Holesovsky <kendy at collabora.com>
AuthorDate: Wed Dec 12 21:03:06 2018 +0100
Commit:     Jan Holesovsky <kendy at collabora.com>
CommitDate: Wed Nov 13 11:56:20 2019 +0100

    sd menubar: Let's not attempt to change the menu description.
    
    It is not a good UX to change names of items in the menu - we have a
    different mean of indicating that the master page view is active: the
    tick in the menu.
    
    Change-Id: I53486847b3929d990de6e398df96b04b943998e6
    Reviewed-on: https://gerrit.libreoffice.org/82498
    Reviewed-by: Jan Holesovsky <kendy at collabora.com>
    Tested-by: Jan Holesovsky <kendy at collabora.com>

diff --git a/loleaflet/src/control/Control.Toolbar.js b/loleaflet/src/control/Control.Toolbar.js
index 563e0f60a..aa9933ee1 100644
--- a/loleaflet/src/control/Control.Toolbar.js
+++ b/loleaflet/src/control/Control.Toolbar.js
@@ -1833,13 +1833,6 @@ function onCommandStateChanged(e) {
 			toolbar.disable('repair');
 		}
 	}
-	else if (commandName === '.uno:SlideMasterPage') {
-		if (state === 'true') {
-			// Rename menu to Close Master and set uno to .uno:CloseMasterView
-		} else {
-			// Rename menu to Master Slide and set uno to .uno:SlideMasterPage
-		}
-	}
 
 	var id = unoCmdToToolbarId(commandName);
 	if (state === 'true') {


More information about the Libreoffice-commits mailing list