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

[Tor Lillqvist (via logerrit)]

 loleaflet/src/control/Control.Menubar.js |    3 +++
 1 file changed, 3 insertions(+)

New commits:
commit f0db3d0d51ad92011f5610ee5c3d44a2a3db2544
Author:     Tor Lillqvist <tml at collabora.com>
AuthorDate: Wed Mar 18 16:54:17 2020 +0200
Commit:     Tor Lillqvist <tml at collabora.com>
CommitDate: Wed Mar 18 16:31:33 2020 +0100

    Add some comments
    
    Change-Id: Ia7153440121acb078f299f79f46fb84d36c67647
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/90704
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice at gmail.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 5759d2e99..add49462f 100644
--- a/loleaflet/src/control/Control.Menubar.js
+++ b/loleaflet/src/control/Control.Menubar.js
@@ -884,11 +884,14 @@ L.Control.Menubar = L.Control.extend({
 
 		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
 		$(function() {
 			var $mainMenuState = $('#main-menu-state');
 			if ($mainMenuState.length) {
 				// animate mobile menu
 				$mainMenuState.change(function() {
+					// This code is invoked when the hamburger menu is opened or closed
 					var $menu = $('#main-menu');
 					var $nav = $menu.parent();
 					if (this.checked) {