-
Notifications
You must be signed in to change notification settings - Fork 130
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
Feature: Multidoor #41
Feature: Multidoor #41
Conversation
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.
Still reviewing but here are some quick comments:
- Not sure if arch did this before, but now there is an extra n-gon behind the arch faces.
FillTypes
for this feel inconsistent. we should consider giving an option for separate fill for door and window.- Also
Components
property UI is too tight, consider using a column with a label and the property. FaceMaps
for this tool not working
@ranjian0 This situation can be fixed cleanly while working on #39 |
That's fine, |
The This could be solved better with a new directory structure. Also, the name |
I will move them to
I could not come up with a better name. You can suggest something? |
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.
Will merge this. Deal with bugs over the weekend.
This is a really good feature. Great Work.
New Feature: Multidoor !
You can now add multiple door/windows under one arch.
Other changes:
fixes #38