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

Internal class to expose engine features #6293

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

mvaligursky
Copy link
Contributor

@mvaligursky mvaligursky commented Apr 26, 2024

related to: #5892, but could be useful many times we need to make the Editor backwards compatible.

@mvaligursky mvaligursky self-assigned this Apr 26, 2024
@mvaligursky mvaligursky requested a review from a team April 26, 2024 12:06
@LeXXik
Copy link
Contributor

LeXXik commented Apr 26, 2024

I think you wanted to link another issue.

@mvaligursky
Copy link
Contributor Author

any thoughts on this one @willeastcott ?

@willeastcott
Copy link
Contributor

I guess I'm not convinced of why this is useful yet...

@kpal81xd
Copy link
Contributor

@mvaligursky was this a more general solution for showing UI components in the editor based on engine version?

@mvaligursky
Copy link
Contributor Author

Yes for UI components. But it might also more important soon as we've decided to keep the Editor backwards compatible with 1.70 - this might play role here.
But happy to wait and see if the Editor actually needs it, which we should see when you start stripping out the WebGL1 support from launcher / publisher @kpal81xd

@willeastcott
Copy link
Contributor

Yeah, can we wait until we need this please? 😄

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

Successfully merging this pull request may close these issues.

None yet

4 participants