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
Make it possible to sort the home screen #7048
Make it possible to sort the home screen #7048
Conversation
# Conflicts: # app/src/main/java/de/danoeh/antennapod/ui/home/HomeFragment.java
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for working on this! I added some comments below :)
app/src/main/java/de/danoeh/antennapod/ui/home/HomeFragment.java
Outdated
Show resolved
Hide resolved
app/src/main/java/de/danoeh/antennapod/ui/home/HomeFragment.java
Outdated
Show resolved
Hide resolved
...rc/main/java/de/danoeh/antennapod/ui/home/settingsdialog/HomeScreenSettingDialogAdapter.java
Outdated
Show resolved
Hide resolved
app/src/main/java/de/danoeh/antennapod/ui/home/settingsdialog/HomeUtil.java
Outdated
Show resolved
Hide resolved
@@ -0,0 +1,5 @@ | |||
package de.danoeh.antennapod.ui.home.settingsdialog; | |||
|
|||
public interface ItemMoveHandler { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about using a BiConsumer here instead of defining a new interface?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried it but unfortunately, BiConsumer
doesn't seem to be available for now.
The regular Consumer
and BiConsumer
from java.util.function
are not possible to use if you want to support Android < 24. For Consumer
, you can instead import it from androidx.core.util
to get it to work. Unfortunately, BiConsumer
doesn't seem to be available in that package. It seems it can be used if you turn on desugaring for the project but I didn't want to do that in this PR.
Or do you have other suggestions here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could create a class with the two positions and use a regular Consumer
that returns an object of that class as well if you think that is better.
ui/preferences/src/main/res/layout/choose_home_screen_order_dialog_entry.xml
Outdated
Show resolved
Hide resolved
ui/preferences/src/main/res/layout/choose_home_screen_order_dialog_header.xml
Outdated
Show resolved
Hide resolved
ui/preferences/src/main/res/layout/choose_home_screen_order_dialog_header.xml
Outdated
Show resolved
Hide resolved
ui/preferences/src/main/res/layout/choose_home_screen_order_dialog_header.xml
Outdated
Show resolved
Hide resolved
ui/preferences/src/main/res/layout/choose_home_screen_order_dialog.xml
Outdated
Show resolved
Hide resolved
# Conflicts: # app/src/main/java/de/danoeh/antennapod/ui/screen/home/HomeFragment.java # app/src/main/java/de/danoeh/antennapod/ui/screen/home/HomeSectionsSettingsDialog.java
Thanks for implementing this! Just a minor comment on the UI: it seems in the screenshot that the 'Hidden' title is bigger than the title of the modal itself, which would be a bit unexpected. Could the 'hidden' be made a bit smaller? |
hey, did you consider #6848? :) |
Hey! I didn't specifically take it into account, but I guess this PR and that one will complement each other really well! 😃 If it is merged before this one, I can fix any conflicts and make sure it works well with those changes. And if this PR is merged first you can do the same. Or do you have other suggestions on what we can do? |
Thanks! Will be released in AntennaPod 3.4.0 |
Description
It is now possible to reorder the sections on the home screen after clicking on "Configure Home Screen". Items can also be hidden if they are dragged down to after the "Hidden" header.
Closes #6068
Checklist