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

Tor Lillqvist (via logerrit) logerrit at kemper.freedesktop.org
Wed Mar 18 19:17:55 UTC 2020


 loleaflet/src/control/Control.Menubar.js |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

New commits:
commit abf7f18d5a1d626cde2a35a595539470b0e62f59
Author:     Tor Lillqvist <tml at collabora.com>
AuthorDate: Wed Mar 18 20:09:25 2020 +0200
Commit:     Tor Lillqvist <tml at collabora.com>
CommitDate: Wed Mar 18 20:17:36 2020 +0100

    Now I understand what that comment meant
    
    Change-Id: I80b420c8f9f4ebf59b9354c435b893a4370bb217
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/90713
    Tested-by: Tor Lillqvist <tml at collabora.com>
    Reviewed-by: Tor Lillqvist <tml at collabora.com>

diff --git a/loleaflet/src/control/Control.Menubar.js b/loleaflet/src/control/Control.Menubar.js
index f44092377..c11e54a5e 100644
--- a/loleaflet/src/control/Control.Menubar.js
+++ b/loleaflet/src/control/Control.Menubar.js
@@ -883,9 +883,7 @@ L.Control.Menubar = L.Control.extend({
 			$('#main-menu').parent().css('height', '0');
 
 		var self = this;
-		// SmartMenus mobile menu toggle button
-		// No idea what the above line means. I am told "smartmenus" is what is used for the NON-mobile "normal"
-		// horizontal menubar. So what is a "SmartMenus mobile menu"? --tml
+		// Also the vertical menu displayed when tapping the hamburger button is produced by SmartMenus
 		$(function() {
 			var $mainMenuState = $('#main-menu-state');
 			if ($mainMenuState.length) {


More information about the Libreoffice-commits mailing list