From 83b4360c368d71f3ead08fdffe8c8422f34dfc28 Mon Sep 17 00:00:00 2001 From: Felix Niklas Date: Tue, 13 Jan 2015 02:19:45 +0100 Subject: [PATCH] add a delay to article meta toggle to allow double click select its 150ms - maybe we need more. plz test. but the fewer delay we need the better cause it'll feel snappier. --- .../app/controllers/ticket_zoom.js.coffee | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/app/assets/javascripts/app/controllers/ticket_zoom.js.coffee b/app/assets/javascripts/app/controllers/ticket_zoom.js.coffee index 8fd4bb0f4..c4e523d1c 100644 --- a/app/assets/javascripts/app/controllers/ticket_zoom.js.coffee +++ b/app/assets/javascripts/app/controllers/ticket_zoom.js.coffee @@ -1262,7 +1262,7 @@ class ArticleView extends App.Controller 'click .show_toogle': 'show_toogle' 'click [data-type=reply]': 'reply' 'click [data-type=replyAll]': 'replyAll' - 'click .text-bubble': 'toggle_meta' + 'click .text-bubble': 'toggle_meta_with_delay' 'click .text-bubble a': 'stopPropagation' constructor: -> @@ -1327,7 +1327,17 @@ class ArticleView extends App.Controller stopPropagation: (e) -> e.stopPropagation() - toggle_meta: (e) -> + toggle_meta_with_delay: (e) => + # allow double click select + # by adding a delay to the toggle + + if @lastClick and +new Date - @lastClick < 150 + clearTimeout(@toggleMetaTimeout) + else + @toggleMetaTimeout = setTimeout(@toggle_meta, 150, e) + @lastClick = +new Date + + toggle_meta: (e) => e.preventDefault() animSpeed = 300