-
-
Notifications
You must be signed in to change notification settings - Fork 384
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
Added the Perspective editor #806
Added the Perspective editor #806
Conversation
I will add them separately
Looks great! 😄
one more smaller change. change |
|
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.
There seems to be an issue with the Line collapsible containers. Expanding the Line1 container in the screenshot, creates another empty collapsible container directly below it, which doesn't seem to do anything. This also happens for every line container, but not in any of the other collapsible containers, like "Lines" or "Point: 1".
I also think that the perspective editor should probably be next to the canvas, but hidden by default, as most people will probably not need it. If you're having difficulties changing the Layout resources, I can do it for you.
Apart from that, I'm not 100% sure about the current status of the UI, but we can make further changes after the PR is merged.
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.
Fantastic work, thank you very much!
This was one of my extensions, i massively improved the original code, and added it to the core so Heavy Testing Encouraged
A Demonstration of Power:
PerspectiveEditor.mp4