Skip to content
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

Jetpack compose #901

Open
OxygenCobalt opened this issue Oct 23, 2024 · 8 comments
Open

Jetpack compose #901

OxygenCobalt opened this issue Oct 23, 2024 · 8 comments
Assignees
Labels
complex This is a complex addition that will take some time ui Related to app UI upgrade Fix deprecation issues or update to new libraries

Comments

@OxygenCobalt
Copy link
Owner

Give up and move literally everything but the nightmarish bottom sheet scaffold to Jetpack Compose.

I'm just tired of the state management issues in the current UI. Working with React during my summer internship and on other projects has been actually enjoyable, compared to the current View model. It would also allow me to do nice little animations easier and actually keep up with Android Studio's rapid deprecation of XML.

I'll try to do it within reason, maintaining the same exact user flow fundamentally and trying to avoid making the app feel "weird". Would guess this co-incides with #761.

@OxygenCobalt OxygenCobalt added upgrade Fix deprecation issues or update to new libraries complex This is a complex addition that will take some time ui Related to app UI labels Oct 23, 2024
@OxygenCobalt OxygenCobalt self-assigned this Oct 23, 2024
@OxygenCobalt OxygenCobalt added rework Requires/Does changes to app architecture and removed rework Requires/Does changes to app architecture labels Oct 23, 2024
@OxygenCobalt OxygenCobalt moved this to Others in Auxio Roadmap Oct 23, 2024
@realjoni17
Copy link

Hi i wanna contribute to it

@OxygenCobalt
Copy link
Owner Author

Cool @realjoni17, although I don't think the app is in a good state for this yet. Compose is somewhat in flux and has some weird UX differences that I would need to hack around to maintain parity with the existing app, so it would be a very delicate and gradual migration.

Additionally, I would also need to do some drastic refactors to the current app architecture just so Compose's idioms actually work, hence #761. Will let you know when the app is good to go for this.

@Koitharu
Copy link
Contributor

I hope this app will not migrate to compose

@OxygenCobalt
Copy link
Owner Author

Hey @Koitharu, do you have anything to share on compose's drawbacks? The most I'm aware of is that it has similar problems to i.e flutter apps, where some components just don't feel quite right. But I think like basic lists and detail views would be able to benefit from compose and they seem pretty native from what I know.

@OxygenCobalt OxygenCobalt moved this from Others to Stabilizing Period in Auxio Roadmap Jan 3, 2025
@navneet851
Copy link

but compose have great upperhand upon xml . due to its flexibility and easy to maintain.

i also wanted to contribute let me know if its possible.

@OxygenCobalt
Copy link
Owner Author

I'm just...hesistant @navneet851. Lots of things in android "look" simple and ergonomic and then crumple into something worse than the "old manual system" once you apply it to something real-world and complex. I have a really high bar for UI stability and if compose doesn't meet it I'm better off with normal XML.

@navneet851
Copy link

once you try it you would never look back , i guarantee it. its that simple and effective.

@navneet851
Copy link

you can watch my song streaming app UI on my github all made upon jetpack compose.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complex This is a complex addition that will take some time ui Related to app UI upgrade Fix deprecation issues or update to new libraries
Projects
Status: Stabilizing Period
Development

No branches or pull requests

4 participants