Skip to content

Commit 4edee85

Browse files
authored
Merge pull request #123 from divyanshugarg36/outside-click
chore: add outside click for menu items fixes issue #124 "Outside click doesn't closes the menu dialog like notification and profile menu"
2 parents 7022cc6 + 14cf171 commit 4edee85

File tree

2 files changed

+15
-2
lines changed

2 files changed

+15
-2
lines changed

imports/ui/components/header/header.js

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,17 @@ import "./header.html";
22
import "./header.scss";
33

44
Template.header.events({
5-
"click #header-profile-button": function() {
5+
"click .main-header": function() {
6+
$("#header-notification-menu").addClass("menu-hidden");
7+
$("#header-profile-menu").addClass("menu-hidden");
8+
},
9+
"click #header-profile-button": function(e) {
10+
e.stopPropagation();
611
$("#header-notification-menu").addClass("menu-hidden");
712
$("#header-profile-menu").toggleClass("menu-hidden");
813
},
9-
"click #header-notification-button": function() {
14+
"click #header-notification-button": function(e) {
15+
e.stopPropagation();
1016
$("#header-profile-menu").addClass("menu-hidden");
1117
$("#header-notification-menu").toggleClass("menu-hidden");
1218
},

imports/ui/layouts/layout.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,9 @@
11
import "./layout.html";
22
import "./layout.scss";
3+
4+
Template.layout.events({
5+
"click #main": function() {
6+
$("#header-notification-menu").addClass("menu-hidden");
7+
$("#header-profile-menu").addClass("menu-hidden");
8+
}
9+
});

0 commit comments

Comments
 (0)