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

Fix terrain brush tools for rotated/scaled terrain #161

Conversation

Dgzt
Copy link
Collaborator

@Dgzt Dgzt commented Apr 28, 2023

Fix for terrain brush tools for rotated/scaled terrain.

  • Up/Down tool
  • Flatten tool
  • Smooth tool
  • Paint tool

@Dgzt Dgzt added the bug Something isn't working label Apr 28, 2023
@Dgzt Dgzt changed the title Fix up/down tool for rotated/scaled map Fix terrain brush tools for rotated/scaled map Apr 28, 2023
@Dgzt Dgzt changed the title Fix terrain brush tools for rotated/scaled map Fix terrain brush tools for rotated/scaled terrain Apr 28, 2023
@Dgzt Dgzt marked this pull request as ready for review April 28, 2023 19:30
@JamesTKhan
Copy link
Owner

Testing this out. I am unable to see the white brush position indicator when editing a rotated terrain though. Is this the same for you? It does not have to be fixed in this PR but just curious if you also have that issue.

@Dgzt
Copy link
Collaborator Author

Dgzt commented May 3, 2023

Testing this out. I am unable to see the white brush position indicator when editing a rotated terrain though. Is this the same for you? It does not have to be fixed in this PR but just curious if you also have that issue.

Yes, it is visible for me.

Peek 2023-05-03 16-57

What are the rotated and scaled properties where you tested it?

@JamesTKhan
Copy link
Owner

I see, I was rotating the terrain vertically. That is a separate issue (not really worried about it) This looks good

@JamesTKhan JamesTKhan merged commit f71f8b5 into JamesTKhan:master May 3, 2023
@Dgzt Dgzt deleted the fix-terrain-brush-tools-on-rotated-or-scaled-terrain branch May 3, 2023 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants