-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Open
Description
@ZornHadNoChoice would like this in in 2.24, I agree. It's currently a developer-only option
Steps remaining IMO:
- Fix a state restore bug: when we select specific notes and move the app to the background where it's killed, when coming back, the dialog will work with an empty notes selection as the browser's ViewModel is a new instance and hasn't yet restored its data. This is problematic because the user is going to change his entire collection while visually just some notes are selected. Fix state restore bug in FindAndReplaceDialogFragment #19981
- Use ViewBinding Convert FindAndReplaceDialogFragment to ViewBinding #19940
- Minor Ui tweaks to make it look more material
- (Optional?)Implement a ViewModel: this would allow to add more testing(my initial tests became flaky, Espresso and our background work don't play too well), potentially makes it easier to fix the above bug

ZornHadNoChoice