From 526f2af69a12e79b433669168b99ece43b7e882c Mon Sep 17 00:00:00 2001 From: Robin Ward Date: Thu, 6 Jun 2013 13:16:36 -0400 Subject: [PATCH] FIX: Touch events conflicted on mobile safari with `{{action}}`s, preventing admin menu option from working. Mobile Safari works with click so this seems to fix it. --- .../javascripts/discourse/views/topic_admin_menu_view.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/assets/javascripts/discourse/views/topic_admin_menu_view.js b/app/assets/javascripts/discourse/views/topic_admin_menu_view.js index 56c65504631..39760f4a72d 100644 --- a/app/assets/javascripts/discourse/views/topic_admin_menu_view.js +++ b/app/assets/javascripts/discourse/views/topic_admin_menu_view.js @@ -9,12 +9,12 @@ Discourse.TopicAdminMenuView = Discourse.View.extend({ willDestroyElement: function() { - $('html').off('mouseup.discourse-topic-admin-menu touchend.discourse-topic-admin-menu'); + $('html').off('mouseup.discourse-topic-admin-menu'); }, didInsertElement: function() { var topicAdminMenuView = this; - return $('html').on('mouseup.discourse-topic-admin-menu touchend.discourse-topic-admin-menu', function(e) { + $('html').on('mouseup.discourse-topic-admin-menu', function(e) { var $target = $(e.target); if ($target.is('button') || topicAdminMenuView.$().has($target).length === 0) { topicAdminMenuView.get('controller').hide();