-
-
Notifications
You must be signed in to change notification settings - Fork 52
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
Comments
Same for joseki explorer. It was not using the same component as thé PvP boards if I remember |
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. |
No description provided.
The text was updated successfully, but these errors were encountered: