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

add EditionDisplay to give human readable names to chapter editions #606

Merged
merged 1 commit into from
Aug 21, 2022

Conversation

robUx4
Copy link
Contributor

@robUx4 robUx4 commented May 22, 2022

When there are multiple editions it should make it easier to pick one for the user.

Otherwise tags with TagEditionUID as the target might be sufficient and more versatile.

Fixes #300

@robUx4 robUx4 added format addition spec_main Main Matroska spec document target matroska-v5 labels May 22, 2022
@hubblec4
Copy link
Contributor

Which edition name(string) should be used if there is an EditionDisplay and a Tag?
Has the TagString from the SimpleTag a higher priority?
Or should the Tag fully ignored if an EditionDisplay exists?

@robUx4
Copy link
Contributor Author

robUx4 commented Jul 31, 2022

Which edition name(string) should be used if there is an EditionDisplay and a Tag?
Has the TagString from the SimpleTag a higher priority?
Or should the Tag fully ignored if an EditionDisplay exists?

I created #655 to fix this issue which also exists for other elements and needs to be handled independently from this PR.

When there are multiple editions it should make it easier to pick one for the
user.

Otherwise tags with TagEditionUID as the target might be sufficient and more
versatile.

Fixes #300
@robUx4
Copy link
Contributor Author

robUx4 commented Aug 21, 2022

Merging this as it's for Matroska v5 and won't affect the specs of Matroska v1 ot v4.

@robUx4 robUx4 merged commit d2a9b21 into master Aug 21, 2022
@robUx4 robUx4 deleted the edition-display branch August 21, 2022 14:03
robUx4 added a commit to Matroska-Org/libmatroska that referenced this pull request Sep 11, 2022
robUx4 added a commit to Matroska-Org/libmatroska that referenced this pull request Sep 11, 2022
robUx4 added a commit to Matroska-Org/libmatroska that referenced this pull request Sep 18, 2022
robUx4 added a commit to Matroska-Org/foundation-source that referenced this pull request Sep 18, 2022
robUx4 added a commit to Matroska-Org/foundation-source that referenced this pull request Sep 18, 2022
robUx4 added a commit to Matroska-Org/libmatroska that referenced this pull request Sep 18, 2022
robUx4 added a commit to Matroska-Org/libmatroska that referenced this pull request Sep 18, 2022
robUx4 added a commit to Matroska-Org/foundation-source that referenced this pull request Oct 2, 2022
robUx4 added a commit to Matroska-Org/foundation-source that referenced this pull request Oct 2, 2022
robUx4 added a commit to Matroska-Org/foundation-source that referenced this pull request Oct 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
format addition matroska-v5 spec_main Main Matroska spec document target
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EditionName
3 participants