-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update the theme to material 3, with new search ui, in addition to other smaller changes). #212
Merged
Conversation
This file contains 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
d1dac6c
to
ed31f97
Compare
caarmen
commented
Jan 2, 2025
app/src/main/kotlin/ca/rmen/android/poetassistant/main/dictionaries/search/Search.kt
Outdated
Show resolved
Hide resolved
caarmen
commented
Jan 2, 2025
app/src/main/kotlin/ca/rmen/android/poetassistant/main/dictionaries/search/Search.kt
Outdated
Show resolved
Hide resolved
caarmen
commented
Jan 2, 2025
...src/main/kotlin/ca/rmen/android/poetassistant/main/dictionaries/search/SuggestionsAdapter.kt
Show resolved
Hide resolved
caarmen
commented
Jan 2, 2025
...c/main/kotlin/ca/rmen/android/poetassistant/main/dictionaries/search/SuggestionsViewModel.kt
Show resolved
Hide resolved
caarmen
commented
Jan 2, 2025
caarmen
commented
Jan 2, 2025
27f52e7
to
a9a3293
Compare
|
335bc3c
to
d6aea0b
Compare
|
|
|
|
|
|
|
de0fa60
to
82dc437
Compare
|
82dc437
to
d598c6c
Compare
|
dfb11ed
to
99c30d3
Compare
|
99c30d3
to
10ec68d
Compare
|
10ec68d
to
bfda09e
Compare
caarmen
commented
Jan 3, 2025
|
bfda09e
to
dcc5548
Compare
|
dcc5548
to
b05477a
Compare
There is an issue with the composer tab: it doesn't scroll? |
|
…abs. This allows us to more easily (and consistently between robolectric + devices) look for elements in the view hierarchy.
… tests. We were setting values for the wrong animation property names.
Remove: * Custom alert dialog themes. * Custom popup menu header color. * Accent color. Replace the play icon with one that has a transparent triangle inside the circle. Adjust tests to find the play button, after the migration to material3 components. Note that the search theming is not correct in this commit. This is addressed in the next commit.
Use the SearchBar and SearchView provided from the material components library, instead of the appcompat one. Some tests are failing. They are addressed in the following commits.
…he material3 search migration widgets.
…e search widget. TODO: Figure out why the app bar is displayed scrolled up initially.
Instead of having this workaround in tests: prevent the issue in the first place. It was being caused by the query word, in the results list header, being selectable. This was making the app bar scroll up just when opening the app, even if there were no queries yet. Only make this `TextView` selectable if it's from an actual user query.
b05477a
to
d57d69f
Compare
…, instead of exiting the app.
… it wasn't possible to scroll to the end of the poem text.
|
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.
Preparation steps
RecyclerView
s in tests: use a unique resource id for the differentRecyclerView
s (rhymer result list, thesaurus result list, ...)UI rework