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

Board theme has no effect while Play Against AI #157

Open
gzyguo opened this issue May 16, 2023 · 3 comments
Open

Board theme has no effect while Play Against AI #157

gzyguo opened this issue May 16, 2023 · 3 comments

Comments

@gzyguo
Copy link

gzyguo commented May 16, 2023

No description provided.

@MathieuDreano
Copy link
Contributor

MathieuDreano commented May 18, 2023

Same for joseki explorer. It was not using the same component as thé PvP boards if I remember

@acristescu
Copy link
Owner

Those two are relics of the old way of writing android apps, which is using view. This app was an early adopters of Compose, back when it was still in Alpha. Even so, the project started back in 2014-15 so not everything is migrated over to Compose.

I'm in the (very slow) process of migrating all the Fragments over to Composables, but it will take some time. I have migrated Settings (not released yet but it's on develop) and I'm in the process on migrating Stats (it's on my laptop).

At some point I'll replace those screens with Compose equivalents. In particular for the AI screen, I'm thinking of just adding AI functionality to the new face-to-face screen and just abandon the old AI screen. The reason is I tried some MVI architecture with it and it seems to create a lot of problems (roughly 80% of issues reported into Firebase are from that screen alone).

This will take a few month though, I'm going to leave this ticket open just in case somebody is impatient and wants to tinker with the old BoardView to make it honor the theme. It shouldn't be hard as the old board view still loads the darker board for dark mode.

@gzyguo
Copy link
Author

gzyguo commented May 18, 2023

Those two are relics of the old way of writing android apps, which is using view. This app was an early adopters of Compose, back when it was still in Alpha. Even so, the project started back in 2014-15 so not everything is migrated over to Compose.

I'm in the (very slow) process of migrating all the Fragments over to Composables, but it will take some time. I have migrated Settings (not released yet but it's on develop) and I'm in the process on migrating Stats (it's on my laptop).

At some point I'll replace those screens with Compose equivalents. In particular for the AI screen, I'm thinking of just adding AI functionality to the new face-to-face screen and just abandon the old AI screen. The reason is I tried some MVI architecture with it and it seems to create a lot of problems (roughly 80% of issues reported into Firebase are from that screen alone).

This will take a few month though, I'm going to leave this ticket open just in case somebody is impatient and wants to tinker with the old BoardView to make it honor the theme. It shouldn't be hard as the old board view still loads the darker board for dark mode.

The AI and free board integration is cool and it's desired from now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants