Award emoji window cannot be closed unless an emoji is chosen
Designs
- Show closed items
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Maintainer
@axil can you please provide some browser info? I can't reproduce with Google Chrome 47.0.2526.111 in OS X 10.11.3
- Author Maintainer
Pff, it's firefox again, chromium works fine :/
Arch Linux / Firefox 43.0.4
@tmaczukin can you check if this happens to you too?
Edited by Achilleas Pipinellis - Achilleas Pipinellis Added ~145894 label
Added ~145894 label
- Maintainer
Yes, I can confirm.
Debian Linux / Firefox (Iceweasel) 39.0
In browser console there is an error:
ReferenceError: event is not defined
. It points to this code:$("html").click(function() { if (!$(event.target).closest(".emoji-menu").length) { if ($(".emoji-menu").is(":visible")) { return $(".emoji-menu").hide(); } } });
the problem is in
!$(event.target)
. I've searched sources and it's generated by this coffeescript: https://gitlab.com/gitlab-org/gitlab-ce/blob/master/app/assets/javascripts/awards_handler.coffee#L9 - Maintainer
I have a MR in for that @tmaczukin !2452 (merged) Same solution. @axil @rdavila Once that is merged it should solve this.
Edited by Jacob Schatz - Contributor
Yes, that MR is a proper fix.
event
variable is not set at all. How did Chrome work? :)I noticed that Chrome predicts common developer's error and just works. It's not first time already.
- Maintainer
@jschatz1 I close this one since it's fixed by !2452 (merged). ;)
- username-removed-128633 Status changed to closed
Status changed to closed