-
Notifications
You must be signed in to change notification settings - Fork 19
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
Reusable layouts below system #269
Comments
In #185 (comment), @joeberkovitz says:
Here's the case for having reusable layout definitions at all levels, not just at the @joeberkovitz again:
and
Its true that repeated "inline" definitions could always be used instead of re-usable layout definitions, but re-usable definitions
On the contrary, I think it would be much easier for manual encoders to use reusable definitions, than for them to have to write the definitions out again every time they are used. Similarly for applications. To me, this feels like an instance of a general principle in programming: Re-use of code is efficient, and leads to better maintainability.
If an application can write a layout definition, it can give it a
If reusable I take your point about recursive constructs being difficult to validate.
I don't see the need for a -definition variant of each reusable element.
This discussion really belongs in #263 (which I have not really been following), but I currently don't understand why there's a fundamental problem here. If there's a clearly defined nesting hierarchy (that includes referenced elements) then every object in the hierarchy has a clearly defined set of styles. We should continue this discussion in #263, when I've caught up there. |
@notator and I have been proposing in #185 (see #185 (comment) for an example) that there should be ”reusable” layout definitions for the layers below
system-layout
, and that these should be siblings ofsystem-layout
.As @joeberkovitz says in #185 (comment) this ends up complicating the definitions, and might not actually be used by the programs making mnx files. And it's not actually necessary for grouping parts, so we're spinning it off into this issue.
The text was updated successfully, but these errors were encountered: