From 4a58a8ece1ab77271005a1528b3dee4ed26d8ff8 Mon Sep 17 00:00:00 2001 From: Colin Lee Date: Wed, 1 May 2019 12:21:13 -0500 Subject: [PATCH] [fenix] For https://github.com/mozilla-mobile/fenix/issues/1870: Copy bookmark folder does nothing (https://github.com/mozilla-mobile/fenix/pull/2193) --- .../org/mozilla/fenix/library/bookmarks/BookmarkAdapter.kt | 3 --- .../org/mozilla/fenix/library/bookmarks/BookmarkItemMenu.kt | 2 +- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkAdapter.kt b/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkAdapter.kt index 31c4fea23..77edc7464 100644 --- a/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkAdapter.kt +++ b/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkAdapter.kt @@ -282,9 +282,6 @@ class BookmarkAdapter(val emptyView: View, val actionEmitter: Observer { actionEmitter.onNext(BookmarkAction.Select(item)) } - is BookmarkItemMenu.Item.Copy -> { - actionEmitter.onNext(BookmarkAction.Copy(item)) - } is BookmarkItemMenu.Item.Delete -> { actionEmitter.onNext(BookmarkAction.Delete(item)) } diff --git a/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkItemMenu.kt b/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkItemMenu.kt index 19995bc6d..eecf49d22 100644 --- a/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkItemMenu.kt +++ b/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkItemMenu.kt @@ -42,7 +42,7 @@ class BookmarkItemMenu( onItemTapped.invoke(BookmarkItemMenu.Item.Select) } } else null, - if (item.type in listOf(BookmarkNodeType.ITEM, BookmarkNodeType.FOLDER)) { + if (item.type == BookmarkNodeType.ITEM) { SimpleBrowserMenuItem(context.getString(R.string.bookmark_menu_copy_button)) { onItemTapped.invoke(BookmarkItemMenu.Item.Copy) }