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

Refine conditional {ex,in}clusion terms. #893

Closed
skynavga opened this issue Jul 15, 2018 · 1 comment
Closed

Refine conditional {ex,in}clusion terms. #893

skynavga opened this issue Jul 15, 2018 · 1 comment

Comments

@skynavga
Copy link
Collaborator

skynavga commented Jul 15, 2018

  • section 2.2

    • [conditionally excluded (or included)] clarify
      that the descendants of a conditionally excluded element are
      all conditionally excluded;

    • [conditionally excluded (or included)] clarify that an element
      is conditionally included if it is not conditionally excluded;

    • introduce sub-term unconditionally included

@nigelmegitt
Copy link
Contributor

nigelmegitt commented Jul 18, 2018

To me this seems a bit too much for the terminology section. The rules are already specified in §8.2.1.

One thing I do notice is that we cannot both exclude all descendants of a conditionally excluded element and except the ignoring of timing semantics.

For example, the timing of an element:

<div timeContainer="seq">
<div condition="false"> <!-- timing is defined by its descendants that are excluded -->
   <p begin="10s" end="15s">
      <span>Some text</span>
   </p>
</div>
<div>
   <p>
      <span>This text should appear at 15s always</span>
   </p>
</div>
</div> <!-- seq container -->

This may need a different issue...

@skynavga skynavga modified the milestones: PR, CR3 Jul 19, 2018
nigelmegitt added a commit that referenced this issue Jul 23, 2018
Improve definition of conditionally excluded (included) (#893).
@skynavga skynavga removed their assignment Jul 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants