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

ZIndex for objects #15228

Open
wants to merge 22 commits into
base: experimental
Choose a base branch
from

Conversation

superEVILFACE
Copy link

This PR adds the zIndex field to objects and turns off dce, which allows for more classes to be used in scripting

@LarryFrosty
Copy link
Contributor

why would you add zIndex whats the benefit

@superEVILFACE
Copy link
Author

well its an easier method of setting the object order 😄

@crowplexus
Copy link
Contributor

why would you add zIndex whats the benefit

easier layering, other game engines do have zIndex and it's generally a better way of setting object order

@crowplexus
Copy link
Contributor

actually can you check if hscript still works? last time this was tried, it completely stopped working altogether

@superEVILFACE
Copy link
Author

working for me
image_2024-08-03_112658180
image_2024-08-03_112711074

@crowplexus
Copy link
Contributor

ah so it should be fine then

@TehPuertoRicanSpartan
Copy link
Contributor

didn't shadowmario say he has problems without dce?

@superEVILFACE
Copy link
Author

pretty sure there used to be problems, but there arent anymore from what i have seen, as it compiles and no errors have come up

@TehPuertoRicanSpartan
Copy link
Contributor

oh alr

@superpowers04
Copy link
Contributor

I'd highly recommend against disabling DCE, DCE prevents Haxe from including cluttered code in the final output for the game, if you need something to be present at runtime, import it somewhere or use @:keep and they'll be forced to compile

@superEVILFACE
Copy link
Author

ight, ill remove those changes and finish fixing stages since zIndex!!!!!

@superEVILFACE superEVILFACE changed the title ZIndex for objects and DCE turned off ZIndex for objects Aug 5, 2024
@DetectiveBaldi
Copy link
Contributor

disabling dce was like the most useful change here 😭

@superEVILFACE
Copy link
Author

they are right though. disabling dce would just bring cluttered code

Copy link

@2JENO 2JENO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool!!!

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

Successfully merging this pull request may close these issues.

7 participants