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

Rewrite to Compose (meta-issue) #478

Closed
31 tasks done
rfc2822 opened this issue Nov 15, 2023 · 5 comments
Closed
31 tasks done

Rewrite to Compose (meta-issue) #478

rfc2822 opened this issue Nov 15, 2023 · 5 comments
Assignees
Labels
refactoring Internal improvement of existing functions

Comments

@rfc2822
Copy link
Member

rfc2822 commented Nov 15, 2023

The idea is to replace XML layouts by Compose.

This should be done gradually and in small, testable steps. For every Activity / Fragment replaced, there should be a separate PR and with every app release, some parts can be replaced (but not everything at once).

The single sub-PRs shouldn't add features or UI enrichment, but provide a minimum Compose replacement for the existing functionality. Ideally,

  • users won't even notice the change,
  • the code is more clean and components are better separated,
  • the PR is easy to test and review,
  • (the number of lines reduces or stays roughly the same)

This is a meta-issue, so it shouldn't get a PR. Every subtask should get one PR. To have a good project flow, please don't change things here or convert tasks to issues yourself.

@ArnyminerZ
Copy link
Member

To have a good project flow, please don't change things here or convert tasks to issues yourself.

Oops, I've just read this. Is there any issue? 😅

@jonatbergn
Copy link

Are you considering compose multiplatform?

@ArnyminerZ
Copy link
Member

Are you considering compose multiplatform?

Why'd you want the app on other platforms? It specifically integrates with Android's Calendar and Contacts provider, which doesn't exist in other OSes. iOS has its own ical support, and in Windows you can use WSA, but there's no such thing as calendar or contacts sharing between apps.

So yeah, right now we are going to stick with Android, however, we are open for suggestions.

Copy link

@rfc2822
Copy link
Member Author

rfc2822 commented Mar 28, 2024

It's done!

@rfc2822 rfc2822 closed this as completed Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Internal improvement of existing functions
Projects
Archived in project
Development

No branches or pull requests

3 participants