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

Migrate to Jetpack Compose (master) #5616

Open
westnordost opened this issue May 5, 2024 · 10 comments
Open

Migrate to Jetpack Compose (master) #5616

westnordost opened this issue May 5, 2024 · 10 comments
Labels
iOS necessary for iOS port

Comments

@westnordost
Copy link
Member

This is the master ticket for migrating from the Android view system (layout XMLs etc.) to Jetpack Compose as UI framework.

The primary reason for this is #5421: To make possible an iOS version of this app that has a shared codebase for the UI through the use of Compose Multiplatform. The first step however is to migrate to Jetpack Compose and only later switch to Compose Multiplatform.

The first foray into Jetpack Compose has been made in PR #5447, I am also new to this but I tried to follow guidelines closely how things should be done, so feel free to look into that PR to learn how the UI code typically looks like before and after.

Contributing

As this is a larger undertaking which can be implemented step by step - this ticket shall serve as a master ticket. Whenever you would like to contribute, post on which screen or view you would like to work on now here and later post a PR when you are done.

Migration is generally done bottom-up, i.e. start with single views first and then work your way up to the screen-level, see Migration Strategy. For converting a whole screen, it is very advisable to do this only after the Fragment or Activity in question has been refactored to use ViewModels, see #5530.

@westnordost westnordost added the iOS necessary for iOS port label May 5, 2024
@westnordost
Copy link
Member Author

westnordost commented May 23, 2024

Current blockers and issues with Compose:

  • Appear-animation missing for achievements and links screen. Lazy item animations not supported by Jetpack Compose yet.

  • Fancy animate-from-icon animation is missing for achievement dialog. Not supported by Jetpack Compose yet

  • (Material2) DatePicker and TimePicker (dialogs) are not available in Jetpack Compose (yet). Necessary for: Logs screen, any quest where a time needs to be set (opening hours, collection times, ...)

  • scroll bars are not supported by Jetpack Compose yet. (They would be handy at least in the Logs screen)

@westnordost
Copy link
Member Author

Re-do of About and Settings screen done, in review. Details here: https://github.com/streetcomplete/StreetComplete/pull/5719/files

@westnordost
Copy link
Member Author

User screens are also done, except for the quest statistics / country statistics screen. See #5595

@westnordost
Copy link
Member Author

I'll look into the two tutorial screens (start tutorial, overlay tutorial) coming week.

@westnordost
Copy link
Member Author

westnordost commented Jul 20, 2024

tutorial screens are done

@westnordost
Copy link
Member Author

Next up would be the buttons and controls on the main screen plus the menus and dialogs there.

So, this is an opportunity to redesign the main menu dialog, which I designed the way it is now chiefly because it was the easiest to do. In the future it could e.g. also be a popup menu, or a sheet that slides in (from the top, from the right?) like the undo menu does... I am open to suggestions. What's popular in other apps?

@mnalis
Copy link
Member

mnalis commented Jul 23, 2024

Other approaches I see:

  • slide from the left - e.g. Osmand (which has menu icon on lower left) and OpenStop, K-9, Libretorrent, Nextcloud, AntennaPo, NewPipe, Fedilab (which have menu icon on upper left corner)
  • slide from the bottom - e.g. Commons App, Vespucci, Organic Maps (which have menu icon on lower right)
  • popup fullscreen - e.g. EveryDoor (which has menu icon on upper left), F-droid (lower right)

I actually quite like current StreetComplete "pop up in middle of the screen" menu, but other approaches are probably fine too, provided they happen fast (i.e. noticeably faster than half a second).

@westnordost
Copy link
Member Author

westnordost commented Aug 5, 2024

Thanks for the input! I ended up mirroring the current appearance now.

I am now working on migrating the UI of the main screen to compose, i.e.

  • (button) controls and star counter, (dropdown on long-press)
  • main menu
  • team mode menu (I plan to redesign it to be a wizard, i.e. similar appearance as the tutorial screens and will commission an artist for some nice illustration)
  • overlay selection dropdown
  • the various messages-dialogs (new-osm-message, "too much to do"-message, achievement unlocked, changelog)
  • upload/download-related dialogs probably
  • location pointer pin: However, I have no idea how to implement that with compose! Maybe this feature has to go! :-( The problem is that the layout of the location pointer pin depends on the layout of the other elements, i.e. it has to be layout after the bounds of the other buttons are known - but in Compose, it is not really possible to inspect that! Help appreciated!

Maybe now, maybe immediately afterwards:

  • edit history sidebar and dialog

@rugk
Copy link
Contributor

rugk commented Aug 16, 2024

slide from the bottom

…generally has the advantage as they can be used with one finger. And IMHO such a panel optionally including a gesture even (moving up from bottom) would be good too in SC, I don't quite like that you e.g. need to press (/go to) the top right icon for opening the menu.

@westnordost
Copy link
Member Author

the complete user screen is now based on compose (including the statistics screen)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
iOS necessary for iOS port
Projects
Status: In Progress
Development

No branches or pull requests

3 participants