Refactor NoteEditor to use changeNoteType dialog#20377
Open
Haz3-jolt wants to merge 3 commits intoankidroid:mainfrom
Open
Refactor NoteEditor to use changeNoteType dialog#20377Haz3-jolt wants to merge 3 commits intoankidroid:mainfrom
Haz3-jolt wants to merge 3 commits intoankidroid:mainfrom
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
Member
Author
Nope! This is only when the noteEditor is opened in edit mode :) Screen_recording_20260303_142948.webm |
Member
Author
|
@david-allison can you restart the lint job I ran ./gradlew ktlintCheck and ./gradlew ktlintFormat locally and couldn't find anything. The job is talking about three resources being unused, but they are actually being used |
Member
|
See: https://github.com/ankidroid/Anki-Android/tree/main/.github/workflows#quality-checks |
- Refactors NoteEditor to use ChangeNoteTypeDialog instead of old implementation
- Since we have migrated to using changeNoteType Dialog in noteEditor we can safely remove a lot of the old code - Unused String and drawable resources will be removed
- Remove unused variable - Remove redundant qualifier name - Explicit type arguments can be inferred
Member
Author
|
@david-allison flaky windows run? |
kevs-mist
added a commit
to kevs-mist/Anki-Android
that referenced
this pull request
Mar 5, 2026
- Add an explicit 5-minute timeout to all runTest calls to prevent UncompletedCoroutinesError on slow Windows runners, where startReviewer() and advanceRobolectricLooper() consume most of the default 1-minute budget - Fix ankiJsUiTest silently not running due to @OverRide instead of @test Fixes ankidroid#20377."
kevs-mist
added a commit
to kevs-mist/Anki-Android
that referenced
this pull request
Mar 5, 2026
- Add explicit 5-minute timeout to all runTest calls - Fix ankiJsUiTest silently not running due to @OverRide instead of @test Fixes ankidroid#20377
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose / Description
Refactors the noteEditor to use
changeNoteTypeDialogand deprecates the old implementationFixes
Approach
How Has This Been Tested?
Pixel Tablet (API 36)
Before:
Screen_recording_20260303_140341.webm
After
Screen_recording_20260303_140105.webm
Checklist
Please, go through these checks before submitting the PR.