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

TETRAEDGE: Support Android and Switch versions #5021

Open
wants to merge 28 commits into
base: master
Choose a base branch
from

Conversation

phcoder
Copy link
Contributor

@phcoder phcoder commented May 18, 2023

Also improves iOS detection/support.

@phcoder phcoder requested a review from mduggan May 18, 2023 14:19
@phcoder phcoder force-pushed the syb-rb branch 4 times, most recently from e496d61 to d881e6b Compare May 18, 2023 19:36
Copy link
Contributor

@mduggan mduggan left a 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.

engines/tetraedge/detection.cpp Outdated Show resolved Hide resolved
engines/tetraedge/obb_archive.cpp Outdated Show resolved Hide resolved
engines/tetraedge/obb_archive.cpp Outdated Show resolved Hide resolved
engines/tetraedge/obb_archive.cpp Show resolved Hide resolved
engines/tetraedge/te/te_core.cpp Outdated Show resolved Hide resolved
engines/tetraedge/te/te_core.cpp Show resolved Hide resolved
engines/tetraedge/te/te_core.cpp Show resolved Hide resolved
Copy link
Contributor

@mduggan mduggan left a 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.

@phcoder phcoder force-pushed the syb-rb branch 2 times, most recently from 8b4adc9 to 5b0eed0 Compare May 23, 2023 10:34
@bluegr
Copy link
Member

bluegr commented May 24, 2023

Nice work!
Since this also adds support for the PS3 versions, I've updated the title of the PR accordingly

@bluegr bluegr changed the title TETRAEDGE: Support Android and Switch TETRAEDGE: Support Android, Switch and PS3 versions May 24, 2023
@phcoder phcoder changed the title TETRAEDGE: Support Android, Switch and PS3 versions TETRAEDGE: Support Android and Switch versions May 24, 2023
@phcoder
Copy link
Contributor Author

phcoder commented May 24, 2023

Nice work! Since this also adds support for the PS3 versions, I've updated the title of the PR accordingly

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

@mduggan
Copy link
Contributor

mduggan commented May 24, 2023

Thanks for the updates! LGTM, I'm happy to merge as long as no other objections to the common code changes.

@lotharsm
Copy link
Member

lotharsm commented Jul 4, 2023

Could you please try to solve the current conflicts so we can merge it? Thanks! :-)

@bluegr
Copy link
Member

bluegr commented Jul 4, 2023

The changes in commit db6567f are done in a cleaner way in commit 9959cc3, so unless I'm mistaken, commit db6567f
shouldn't be needed anymore

@phcoder any feedback on this?

@bluegr
Copy link
Member

bluegr commented Nov 6, 2023

@phcoder any updates on this?

There are a lot of conflicts now, which need to be addressed

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