5 Commits (88765c348bd4d07a207e159bea6721ebf03d89f6)

Author	SHA1	Message	Date
ValentinTimisica	03284e2da0	Fixes #10674: Add SignInView only once	4 years ago
Grisha Kruglov	853a0acab4	Closes #4046: Always display 'Desktop' bookmarks folder ... See https://github.com/mozilla-mobile/fenix/issues/4046 for a detailed discussion of this. In short, this patch removes code that would conditionally hide desktop bookmarks depending on the signed-in state of the browser.	4 years ago
Tiger Oakes	ee1f040e53	No issue: Move signedIn tracking to BookmarksSharedViewModel (#4755)	5 years ago
Mugurell	645674c9bd	Closes #4396 - Add a Bookmarks Controller (#4593) ... * For #4396 - Rename BookmarkInteractor methods Following the naming model used in other Interactors this too will use reactive method names in the form of "on..." instead of the previous imperative model. Kept the imperative naming model for the methods from `SelectionInteractor` as they are a new addition and I'm not sure about the future direction. * For #4396 - Add a BookmarkController It abstracts the Fragment behavior in a contract through which various Interactors can inform about the specific View changes and can ask for modifications in their container Fragment. This contract and it's implementation - `DefaultBookmarkController` are the result of extracting the container Fragment's business logic from `BookmarkFragmentInteractor` in it's own standalone component. * For #4396 - Refactored Bookmark related tests Added a new `BookmarkControllerTest` tests class which complements the new `BookmarkController` to ensure that it properly operates on `BookmarkFragment` Also refactored the existing `BookmarkFragmentInteractorTest` to accommodate `BookmarkFragmentInteractor`'s now more specialized behavior.	5 years ago
Colin Lee	3dc20543e3	For #4128: Migrate Bookmarks to LibState (#4254)	5 years ago