-
-
Notifications
You must be signed in to change notification settings - Fork 999
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
TETRAEDGE: Support Android and Switch versions #5021
base: master
Are you sure you want to change the base?
Conversation
e496d61
to
d881e6b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good - thanks for fixing the file search system, just left a few comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good - thanks for fixing the file search system, just left a few comments.
8b4adc9
to
5b0eed0
Compare
This allows the languages to be modified after initial detection as a result of e.g. installing language patch
FSNode doesn't work for archives. And we need to use .obb in packed Android version.
Fortunately it's not compressed, so we don't have to keep a copy in memory
This allows ot e.g. detect if Hebrew patch was applied after initial detection
This is a minor tag used on switch but without declaration we error-out. Just skip it for now
Nice work! |
It doesn't yet. Menu doesn't show up properly and if I bypass the menu, then this version is missing the mouse cursor and hence doesn't show it which breaks the experience |
Thanks for the updates! LGTM, I'm happy to merge as long as no other objections to the common code changes. |
Could you please try to solve the current conflicts so we can merge it? Thanks! :-) |
@phcoder any updates on this? There are a lot of conflicts now, which need to be addressed |
Also improves iOS detection/support.