Skip to content

Conversation

@SteveLLamb
Copy link
Member

@SteveLLamb SteveLLamb commented Jun 30, 2023

Add (Informative) text to clauses when flagged with data-informative="true"

@SteveLLamb SteveLLamb requested a review from palemieux June 30, 2023 22:54
@github-actions
Copy link

github-actions bot commented Jun 30, 2023

@SteveLLamb SteveLLamb marked this pull request as draft July 1, 2023 16:54
@SteveLLamb SteveLLamb marked this pull request as ready for review July 7, 2023 17:07
@SteveLLamb
Copy link
Member Author

This is per request on late comments for editorials updates on a DP ballot by @ERyan71258

@SteveLLamb SteveLLamb requested a review from ERyan71258 May 16, 2025 17:03
Copy link
Member

@palemieux palemieux left a comment

Choose a reason for hiding this comment

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

Informative elements should be indicated using class="informative" just as appendices are signaled using class="appendix".

@SteveLLamb
Copy link
Member Author

Informative elements should be indicated using class="informative" just as appendices are signaled using class="appendix".

update to using class in 0da2ce0

@SteveLLamb SteveLLamb requested a review from palemieux May 16, 2025 17:34
@SteveLLamb SteveLLamb changed the title Informative Clauses Informative Annexes May 16, 2025
@ERyan71258
Copy link
Collaborator

Would an editor also be able to add "(Normative)" to an annex title? Per ISO, an annex must be specified as either Normative or Informative.

@SteveLLamb
Copy link
Member Author

SteveLLamb commented May 16, 2025

Would an editor also be able to add "(Normative)" to an annex title? Per ISO, an annex must be specified as either Normative or Informative.

@ERyan71258 I changed the logic to default to putting "(Normative)" unless declared "informative" by the editor in 1b7f1c6, as noted in https://doc.smpte-doc.org/html-pub/feature/infomative-clause-text/#sec-annex-section

@SteveLLamb SteveLLamb merged commit dca1e12 into main May 19, 2025
2 checks passed
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.

4 participants