-
Notifications
You must be signed in to change notification settings - Fork 33
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
Generators/generic brush editing #132
Generators/generic brush editing #132
Conversation
…ndles multiple times
…rately from the handling the control itself
…rializeable and will not be stored :(
… been selected/deselected
…r with an invalid brush
…ices is not initialized
Tested and works well, but not done a code review |
As I have been using this branch ever since I started on the source importer I can confirm that it works well. I gave the code a read, looks good. However the public API to create brushes is too convoluted for newcomers. That doesn't have to be addressed in this PR but it's something to keep in mind. This could be mitigated through better naming, more IntelliSense documentation or alternatively a wrapper class that makes common operations easier to prevent them working too low-level. I am barely able to follow the code I wrote myself just to make a box and clip some sides. |
Added support for generic brush editing (issue #86)