Matthew Finkel pushed to branch tor-browser-82.0.0b4-10.0-1 at The Tor Project / Applications / fenix
Commits:
-
d0ee6fe3
by Alex Catarineu at 2020-10-22T13:25:22+02:00
-
8c969c0c
by Matthew Finkel at 2020-10-22T22:18:33+00:00
1 changed file:
Changes:
| ... | ... | @@ -48,7 +48,6 @@ import org.mozilla.fenix.ext.bookmarkStorage |
| 48 | 48 |
import org.mozilla.fenix.ext.components
|
| 49 | 49 |
import org.mozilla.fenix.ext.minus
|
| 50 | 50 |
import org.mozilla.fenix.ext.nav
|
| 51 |
-import org.mozilla.fenix.ext.requireComponents
|
|
| 52 | 51 |
import org.mozilla.fenix.ext.toShortUrl
|
| 53 | 52 |
import org.mozilla.fenix.library.LibraryPageFragment
|
| 54 | 53 |
import org.mozilla.fenix.utils.allowUndo
|
| ... | ... | @@ -129,16 +128,9 @@ class BookmarkFragment : LibraryPageFragment<BookmarkNode>(), UserInteractionHan |
| 129 | 128 |
@ExperimentalCoroutinesApi
|
| 130 | 129 |
override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
|
| 131 | 130 |
super.onViewCreated(view, savedInstanceState)
|
| 132 |
- val accountManager = requireComponents.backgroundServices.accountManager
|
|
| 133 | 131 |
consumeFrom(bookmarkStore) {
|
| 134 | 132 |
bookmarkView.update(it)
|
| 135 |
- |
|
| 136 |
- // Only display the sign-in prompt if we're inside of the virtual "Desktop Bookmarks" node.
|
|
| 137 |
- // Don't want to pester user too much with it, and if there are lots of bookmarks present,
|
|
| 138 |
- // it'll just get visually lost. Inside of the "Desktop Bookmarks" node, it'll nicely stand-out,
|
|
| 139 |
- // since there are always only three other items in there. It's also the right place contextually.
|
|
| 140 |
- bookmarkView.view.bookmark_folders_sign_in.isVisible =
|
|
| 141 |
- it.tree?.guid == BookmarkRoot.Root.id && accountManager.authenticatedAccount() == null
|
|
| 133 |
+ bookmarkView.view.bookmark_folders_sign_in.isVisible = false
|
|
| 142 | 134 |
}
|
| 143 | 135 |
}
|
| 144 | 136 |
|