diff options
author | stefan-niedermann <info@niedermann.it> | 2020-01-23 15:05:54 +0300 |
---|---|---|
committer | Niedermann IT-Dienstleistungen <stefan-niedermann@users.noreply.github.com> | 2020-01-23 15:07:00 +0300 |
commit | 520be1710da4627ae441138dbde4ea3b69b3eb68 (patch) | |
tree | e9dadd7190592022c9626f051bade615f98acf14 | |
parent | 63553fb6fc6dca24f0c7bfea9641dbb44da116a3 (diff) |
#550 In-note-search doesn't jump to occurrence of searchstring
- Reduce calls of getLayout() for performance reasons
-rw-r--r-- | app/src/main/java/it/niedermann/owncloud/notes/android/fragment/SearchableBaseNoteFragment.java | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/app/src/main/java/it/niedermann/owncloud/notes/android/fragment/SearchableBaseNoteFragment.java b/app/src/main/java/it/niedermann/owncloud/notes/android/fragment/SearchableBaseNoteFragment.java index fcc2c718..35b17a17 100644 --- a/app/src/main/java/it/niedermann/owncloud/notes/android/fragment/SearchableBaseNoteFragment.java +++ b/app/src/main/java/it/niedermann/owncloud/notes/android/fragment/SearchableBaseNoteFragment.java @@ -169,7 +169,8 @@ public abstract class SearchableBaseNoteFragment extends BaseNoteFragment { } private void jumpToOccurrence() { - if (getLayout() == null) { + Layout layout = getLayout(); + if (layout == null) { Log.w(TAG, "getLayout() is null"); return; } @@ -199,10 +200,10 @@ public abstract class SearchableBaseNoteFragment extends BaseNoteFragment { return; } String textUntilFirstOccurrence = currentContent.substring(0, indexOfNewText); - int numberLine = getLayout().getLineForOffset(textUntilFirstOccurrence.length()); + int numberLine = layout.getLineForOffset(textUntilFirstOccurrence.length()); if (numberLine >= 0) { - getScrollView().smoothScrollTo(0, getLayout().getLineTop(numberLine)); + getScrollView().smoothScrollTo(0, layout.getLineTop(numberLine)); } } |