From f3ea4a4f45daa41d0aa032b1dc4804a5f6ff503c Mon Sep 17 00:00:00 2001 From: ozero dien <139508+ozero@users.noreply.github.com> Date: Thu, 9 Jan 2020 13:59:56 +0900 Subject: [PATCH 1/2] Stop scrollToItem() animate twice. - Suppress first fade-to-black with `animate(colWhite, 1)` - Stop animate twice in `colShiny` with `).promise().done(` --- view/theme/frio/js/theme.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/view/theme/frio/js/theme.js b/view/theme/frio/js/theme.js index 46cea59ba2..2ace5f9573 100644 --- a/view/theme/frio/js/theme.js +++ b/view/theme/frio/js/theme.js @@ -650,9 +650,9 @@ function scrollToItem(elementId) { // Scroll to the DIV with the ID (GUID) $('html, body').animate({ scrollTop: itemPos - }, 400, function() { + }, 400).promise().done( function() { // Highlight post/commenent with ID (GUID) - $el.animate(colWhite, 1000).animate(colShiny).animate(colWhite, 600); + $el.animate(colWhite, 1).animate(colShiny,200).animate(colWhite, 600); }); } From 0a5b3a2bc719c4e010c0d6ed138a1669e618321b Mon Sep 17 00:00:00 2001 From: ozero dien <139508+ozero@users.noreply.github.com> Date: Wed, 15 Jan 2020 18:58:00 +0900 Subject: [PATCH 2/2] Update theme.js Reverted the animation change. --- view/theme/frio/js/theme.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/view/theme/frio/js/theme.js b/view/theme/frio/js/theme.js index 2ace5f9573..f1bef07f20 100644 --- a/view/theme/frio/js/theme.js +++ b/view/theme/frio/js/theme.js @@ -652,7 +652,7 @@ function scrollToItem(elementId) { scrollTop: itemPos }, 400).promise().done( function() { // Highlight post/commenent with ID (GUID) - $el.animate(colWhite, 1).animate(colShiny,200).animate(colWhite, 600); + $el.animate(colWhite, 1000).animate(colShiny).animate(colWhite, 600); }); }