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

Disable properties that should not be used (scale/rotation) (or warn the user) #32975

Open
fabriceci opened this issue Oct 22, 2019 · 1 comment

Comments

@fabriceci
Copy link
Contributor

fabriceci commented Oct 22, 2019

Godot version: 3.2 alpha 2
OS/device including version: macOS Mojave
Issue description:
I faced the same issue as #5929, reduz said it's not an issue but a limitation of the physic engine, scale must not be used with this node (or a rigid in non static mode) and the issue was closed.

This issue is similar but with the rotation property #2129

However those fields are accessible in the UI/Code and nothing warn the user of that situation.

I suggest if possible to disable those properties if they can't be used or at least display a warning/error message to avoid users to face the problem.

@fabriceci fabriceci changed the title move() turns KinematicBody2D's scale from negative to positive Disable properties that should not be used (scale/rotation) (or warn the user) Oct 22, 2019
@chucklepie
Copy link

Hello, so how exactly are you supposed to flip all the raycasts (and others) you have? this isn't usually just a sprite you're changing, but a dozen nodes inside a kinematic body? Do you expect everyone to have double the number of nodes and inside code switch between the two, or perhaps every time you want to switch, do the maths and reverse every ray and collision box...

At least implement the above and disable the node, cos after 3 hours of coding and testing, I just realised I've completely wasted my time :(

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

No branches or pull requests

3 participants