From c452152f74507cb5fd333b9f8e48e61d8666437b Mon Sep 17 00:00:00 2001 From: Marcin Kurczewski Date: Sat, 18 Oct 2014 09:59:34 +0200 Subject: [PATCH] Removed tag input focusing in post upload This caused multiple problems with too eager scrolling: - adding a new post to upload queue scrolled the site to the tag input - problems on android, where keyboard can change viewport size on focus --- TODO | 1 - public_html/js/Presenters/PostUploadPresenter.js | 1 - 2 files changed, 2 deletions(-) diff --git a/TODO b/TODO index 706150e5..58124ade 100644 --- a/TODO +++ b/TODO @@ -3,7 +3,6 @@ first major release. - autocomplete: don't reset active item if suggestions stay the same - autocomplete: don't show items that are already used in tag list (unsure) -- posts/upload: fix scrolling due to focus when selecting multiple posts - posts/upload: when adding multiple files, select first one, not last one - posts/upload: better hotkeys for going to next post - posts/upload: get rid of suggested tags after selecting next post diff --git a/public_html/js/Presenters/PostUploadPresenter.js b/public_html/js/Presenters/PostUploadPresenter.js index 29eb719d..634bd6b9 100644 --- a/public_html/js/Presenters/PostUploadPresenter.js +++ b/public_html/js/Presenters/PostUploadPresenter.js @@ -341,7 +341,6 @@ App.Presenters.PostUploadPresenter = function( hidePostEditForm(); } else { showPostEditForm(selectedPosts); - tagInput.focus(); } $el.find('.post-table-op').prop('disabled', selectedPosts.length === 0); }