Commit Graph

122 Commits (21df5a1a4ae8f94bd6ad4eb3a68ddcf57635f61c)

Author SHA1 Message Date
Chenxia Liu 41ee8a4efb For #6294 - When only one tab is being saved to collection, add it to selected tabs. 5 years ago
Severin Rudie aa8642f534
#4596 migrate collections (#5911)
* For #4596: move code from CollectionCreationComponent to CollectionCreationStore

Other than adding comments, no changes were made. The code will be updated in a following commit. This is in order to make the commit diff more readable.

* For 4596: update CollectionCreateStore to libstate

* For 4596: copied CollectionCreationUIView into CollectionCreationView

Otherwise, no code was changed. The next commit will update this code. This is in order to make the commit diff more readable.

* For 4596: update CollectionCreationView to LibState

Note that the minimal changes possible to enable migration were made.  Refactoring will happen in a later commit.

* For 4596: updated CollectionCreationTabListAdapter to work with the new View

* For 4596: updated SaveCollectionListAdapter to work with the new View

* For 4596: implemented CollectionCreationController

For now, it has an identical interface to the interactor. In a later commit several of its responsibilities will be moved around, some to the interactor and some to the reducer

* For 4596: copied over previous reducer code

No other changes were made. The code will be updated in the following commit. This is done to make changes more readable for the reviewer

* For 4596: update reducer code param names

Otherwise, no changes at this time

* For 4596: add arguments to CreateCollectionFragment in nav_graph

These will be used to replace the current CreateCollectionViewModel, which shares data between fragments in a way that doesn't fit within our architecture.

* For 4596: pass arguments to collection via transaction instead of VM

The VM will be removed in a later commit

* For 4596: update BrowserToolbarController to share state to collection via its Direction

* For 4596: removed CreateCollectionViewModel

* For 4596: test tab retrieval in CreateCollectionFragment

* For 4596: fix crashing CreateCollectionFragmentTest

* For 4596: removed classes create collection classes used by old architecture

* For 4596: collection interactor rename + kdoc

* For 4596: moved collection interactor interface

* For 4596: renamed CreateCollectionFragment

All related classes followed the pattern of CollectionCreationX

* For 4596: kdoc CollectionCreationController

There's no effective difference between these calls and their interactor equivalent, so I linked to them

* For 4596: fix bug that caused rename to not work

* For 4596: removed unused collection actions

These were unused before the LibState refactor

* For 4596: kdoc StepChanged

* For 4596: removed todos about moving logic to the reducer

saveTabsToCollection: this could be moved, but that would involve creating a new action. SaveCollectionStep should probably be refactored out, so adding this layer of indirection seemed counterproductive

handleBackPress: needs to be able to call dismiss(). The reducer doesn't (and shouldn't) be able to do that, so this needs to live here

stepBack: called by handleBackPress. See above

* For 4596: wrote tests for CollectionCreationController#stepback

* For 4596: fixed tests broken by changes to collections

* For 4596: small readability refactor for CollectionController#stepBack

No change to functionality (see tests)

* For 4596: broke apart CollectionView#update

There's probably a lot more that could be done here, but smaller changes were made to reduce scope

* For 4596: remove unnecessary todos

It looks like we don't follow the suggested pattern in this project

* For 4596: test CollectionCreationController#normalSessionSize

* For 4596: updated naming in CollectionCreationController per review
5 years ago
mihai-adrian ef4a6b4154 For #4877: Restored logic for closing tabs when not on browserFragment
Added return in function when saving collections coming from browser fragment
5 years ago
Colin Lee 709588572a
For #5055: Crash when opening bookmarks (#5058) 5 years ago
Jeff Boek 68f5d211d3 For #4326 - Updates codebase to support latest version of detekt 5 years ago
Tiger Oakes ac6c1ec2ee Closes #2690 - Clean up CreateCollection viewmodel (#4731) 5 years ago
Sawyer Blatz 4731977f5d
For #4421: Fixes issues with new glean events (#4816) 5 years ago
Jorge De Los Santos 337ad1e88e For #3424 - Make talkback ignore ImageButton in Collection Sele… (#3792)
For #3424 - Make talkback ignore ImageButton in Collection Selection by removing contentDescription and setting importantForAccessibility=no
5 years ago
Colin Lee ef97173cd9
For #4529, #4427: Resuming after restoring instance state breaks UI (#4571)
* For #4529, #4427: Resuming after restoring instance state breaks UI

* Clear up warnings
5 years ago
Sawyer Blatz 9d65e665e5 No issue: Removes deprecated usage of ViewModelProviders 5 years ago
Nikit Bhandari 671727c3e8 For #4341 use camelCase for views referenced from code 5 years ago
Nikit Bhandari 46de06d132 For #4339 reduce build warnings (#4340) 5 years ago
mihai-adrian 458632f4bf fixes #4237 - Cannot edit the default name of a collection, only replace it
changed setting of text to be done before transition, preventing transition to make textView unclickable;
 added renaming situation to showKeyboard in onResume();
 changed title text when renaming, from: "Name collection" to "Rename collection".
5 years ago
mihai-adrian ef3c813aee fixes #4237 - Cannot edit the default name of a collection, only replace it
changed setting of text to be done before transition, preventing transition to make textView unclickable;
 added renaming situation to showKeyboard in onResume();
 changed title text when renaming, from: "Name collection" to "Rename collection".
5 years ago
mihai-adrian 2e300fc8cf fixes #4237 - Cannot edit the default name of a collection, only replace it
made name_collection_edittext field clickable
5 years ago
Tiger Oakes 88c05a5f43 Use glide-like image loading function 5 years ago
Nikit Bhandari 9d9685625a For #4201 fix app crash on renaming a collection (#4202)
* For #4201 fix app crash on renaming a collection

* update
5 years ago
Sawyer Blatz 13198f56df For #969: Adds telemetry for collections (#3935) 5 years ago
Tiger Oakes 6da6ddb095 Use StateListDrawable & resources w/ TabCollection 5 years ago
Tiger Oakes ef8d9604ae No issue: Normalize license header comment (#3909) 5 years ago
Tiger Oakes 8cd1a0cb38 Move adapter Job setup code to helper class (#3407) 5 years ago
Sawyer Blatz 7e46e71d00 For #3005: Closes tabs when saving to collection (#3723) 5 years ago
Tiger Oakes 4994554576 No issue: Adds lint exception for when statement (#3685) 5 years ago
Tiger Oakes 30930cdc88 Import variables referred to by path 5 years ago
Tiger Oakes d01bb8c442 Use lifecycleScope for collections fragments 5 years ago
vmbaicu 021900e7ad Fixes #3527 - Set Collection name max length (#3531) 5 years ago
Nikit Bhandari cfefc5632b For #3489: Disables blank collection names (#3490) 5 years ago
Emily Kager 6bd9b82385 No issue: Remove flash of tab adapter updating data in create collection 5 years ago
Emily Kager e28277693b Closes #3428 - Make + in collection creation selection stage clickable 5 years ago
Emily Kager 8b4c2db4e4 For #3433 - Add checkbox hide changes to change payload 5 years ago
Emily Kager edc1422b68 For #3288 - call setChecked after calling setOnCheckedChangeListener (#3357) 5 years ago
Sawyer Blatz decacbfc97 For #3323 Runs PublicSuffixList synchronously 5 years ago
Nikit Bhandari d814e95794 fix crash on renaming a collection 5 years ago
Sawyer Blatz 45c509fbce
For #2648: Updates trimming of url hostname (#3191)
* For #2648: Updates trimming of url hostname

* For #2648: Fix nits
5 years ago
Emily Kager 7368a95d29 No issue: Sets up observers for tab collection changes (#3248) 5 years ago
ekager 87f6268816 For #3101 - Do not show tab selection screen for 1 tab 5 years ago
Emily Kager 0327b1146b For #2977 - Update add new collections flow (#2991)
* For #2977 - Update add new collections flow

* Rename shared elements to be more general

* Make tab list not clickable in other modes

* For #2577 - Stop Flickering in List

* Add extensions function for next step with collections list size
5 years ago
Sawyer Blatz 9e0e5d95d9 For #2931: Show browser snackbar above quick action sheet 5 years ago
Sawyer Blatz 6a53127ef9 For #2948: Adds menu button add tab to collection 5 years ago
Emily Kager 1ef85433b4 For #2891 - Make checkbox invisible so text isn't cut off 5 years ago
Emily Kager a6a72885d2 For #2898 - Adds Deselect All in Collection Creation Select Tabs Step 5 years ago
Harsh Shandilya 7d9c8726c5 treewide: Resolve compilation warnings (#2859)
Signed-off-by: Harsh Shandilya <msfjarvis@gmail.com>
5 years ago
Emily Kager 01c9317be0 For #2843 - Improve Collections UI and UX (#2851) 5 years ago
Sawyer Blatz 549292d06e For #2732: Removes view model usage from collection creation 5 years ago
Sawyer Blatz 87df0a8103 For #2728 & #2729: Uses view model selected tabs for collection creation (#2748) 5 years ago
Sawyer Blatz 93d0982a6f For #1576: Adds tab collection renaming 5 years ago
Sawyer Blatz afbe397f94 For #1575: Enables adding tabs to existing collection 5 years ago
Sebastian Kaspari 291df48c4e Install the "icons" WebExtension to automatically load icons for every visited website. 5 years ago
Sawyer Blatz 7861a80715 For #2205: Truncates tab collection titles 5 years ago
Sawyer Blatz 4e6f9b9ef1 For #1574: Displays previous tab collections when creating 5 years ago
Sawyer Blatz 60e5f8a626 For #2351: Fixes backspace key ignored in collections 5 years ago
Sawyer Blatz 30a643720f For #1574: Adds tab collection creation 5 years ago
Sawyer Blatz 72d29c2a43
For #2205 & #1578: Integrates tab collection storage (#2478)
* For #2205: Adds TabCollectionStorage

* For #1578: Adds delete to TabCollection
5 years ago
Jeff Boek c5e5ef4b25
Teases apart ViewModel dependencies (#2499)
* No Issue - pulls render outside of the viewmodel

* No Issue - Properly subscribes to the changesObservable

* No Issue - Fixes ViewModel tests
5 years ago
Colin Lee 49ac62ab85 Fix ViewModel States (#2457)
Co-authored-by: Jeff Boek <jeff@jeffboek.com>
5 years ago
Emily Kager 4070941657 No issue: Refactor dialogs to use nav graph 5 years ago
Colin Lee ccbc14a71f For #1994: Re-architect state handling code (#2382) 5 years ago
Sawyer Blatz 7d577e5953
For #2205: Adds collections view to home fragment (#2249)
* For #1574: Adds collections to home view

* Adds colored icons and expansion

* Adds state change

* Adds more styling

* Adds ItsNotBrokenSnacks

* Adds chevron

* Improves styling of swipe to delete and adds delete action

* Fix nits

* Try to add real saving
5 years ago
Emily Kager 7dda013ad1 No issue: Remove plurals strings (#2234) 5 years ago
Emily Kager 2953b54a84 For #1573 - Polish Collections UI Component (#2212)
* Adds Keyboard resuming, Snackbar verification, layout edits to collections

Adds Keyboard resuming, Snackbar verification, layout edits to collections

* Adds new strings for collections/tabs management

* Adds constraintsets, hides checkboxes, adds scrim

* Update strings to plurals
5 years ago
Emily Kager 34f4720e6b For #1573 - Create Save to Collection Component and Views 5 years ago
Emily Kager 1041500869 For #1573 - Long pressing tab selects that tab in save collections 5 years ago
Jeff Boek 160e5ea1c8 For #1843 - Wires up add to collection button 5 years ago
Jeff Boek f58400660a For #1843 - Wires up close button 5 years ago
Jeff Boek cddb4de79b For #1843 - Adds the selected tabs button 5 years ago
Jeff Boek dea169dca2 For #1843 - Adds ability to select all tabs 5 years ago
Jeff Boek ff08d0dbb0 For #1843 - Passes selected tabs through the MVI loop 5 years ago
Jeff Boek 9df64b61d4 For #1843 - Closes the fragment when the arrow button is pressed 5 years ago
Jeff Boek d39c15402e For #1843 - Adds a better theme for the creation fragment 5 years ago
Jeff Boek ba90b58d32 For #1843 - Displays tabs on the tab selection screen 5 years ago
Jeff Boek ab4cf786d7 For #1843 - Scaffolds MVI component for Collection Creation 5 years ago
Jeff Boek 63574cc359 For #1843 - Adds a create collections fragment and navigates to it 5 years ago