Skip to content

Add open questions#272

Merged
Robbendebiene merged 27 commits intoOPENER-next:masterfrom
wielandb:open-questions
Jul 18, 2024
Merged

Add open questions#272
Robbendebiene merged 27 commits intoOPENER-next:masterfrom
wielandb:open-questions

Conversation

@wielandb
Copy link
Copy Markdown
Contributor

@wielandb wielandb commented Jul 1, 2024

This PR adds questions regarding the recent open= proposal.

Theese questions are:

  • Is this door normally open?
  • How wide is the opening in this partially open door?

@wielandb wielandb marked this pull request as ready for review July 2, 2024 10:14
@7h30n3
Copy link
Copy Markdown
Member

7h30n3 commented Jul 3, 2024

I also think the tag doesn't make much sense for sliding doors. So they should be excluded as well.

@wielandb
Copy link
Copy Markdown
Contributor Author

wielandb commented Jul 9, 2024

I also think the tag doesn't make much sense for sliding doors. So they should be excluded as well.

Really? One of our examples in the proposal is a sliding door.

@wielandb
Copy link
Copy Markdown
Contributor Author

Ready for re-review now, schematic images are to be added later.

@wielandb
Copy link
Copy Markdown
Contributor Author

Should be ready for re-review now

@wielandb wielandb requested a review from Robbendebiene July 16, 2024 10:48
@wielandb wielandb requested a review from Robbendebiene July 18, 2024 09:01
Copy link
Copy Markdown
Member

@Robbendebiene Robbendebiene left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚪

@Robbendebiene Robbendebiene merged commit 8f7fdc2 into OPENER-next:master Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants