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

Clarify that the scheme attribute does not have a default vocabulary. #2382

Closed
mattgarrish opened this issue Jul 29, 2022 · 2 comments · Fixed by #2379
Closed

Clarify that the scheme attribute does not have a default vocabulary. #2382

mattgarrish opened this issue Jul 29, 2022 · 2 comments · Fixed by #2379
Labels
EPUB33 Issues addressed in the EPUB 3.3 revision Spec-EPUB3 The issue affects the core EPUB 3.3 Recommendation Topic-PackageDoc The issue affects package documents

Comments

@mattgarrish
Copy link
Member

It's the only attribute that requires a prefix for its values.

@iherman
Copy link
Member

iherman commented Jul 30, 2022

Indeed... I was surprised when I saw that in the PR on processing property, I didn't remember that particularity...

@dauwhe dauwhe added the Agenda+ Issues that should be discussed during the next working group call. label Aug 3, 2022
@iherman
Copy link
Member

iherman commented Aug 5, 2022

The issue was discussed in a meeting on 2022-08-04

List of resolutions:

View the transcript

2. Re-implement vocabulary association mechanisms as an algorithm (pr epub-specs#2379)

See github pull request epub-specs#2379.

See github issue epub-specs#2378.

See github issue epub-specs#2382.

Dave Cramer: there are a couple issues associated with this (#2378, #2382).
… this is very technical, I wish mgarrish were here to explain.
… doesn't really seems to change how the spec works, more an editorial change.
… seems like they were just looking for WG approval before merging, even though most of us probably won't have strong opinions on this.
… i'm generally in favor of explaining things in an algorithmic fashion rather than hard to understand paragraphs.
… maybe we just say that we don't have objection to merging this pending github reviews?.

Proposed resolution: The WG supports merging #2379 pending approval from the assigned reviewers.. (Dave Cramer)

Dave Cramer: +1.

Matthew Chan: +1.

Brady Duga: +1.

Toshiaki Koike: +1.

Shinya Takami (高見真也): +1.

Resolution #1: The WG supports merging #2379 pending approval from the assigned reviewers..

@mattgarrish mattgarrish added EPUB33 Issues addressed in the EPUB 3.3 revision and removed Agenda+ Issues that should be discussed during the next working group call. labels Aug 10, 2022
@mattgarrish mattgarrish added the Spec-EPUB3 The issue affects the core EPUB 3.3 Recommendation label Sep 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EPUB33 Issues addressed in the EPUB 3.3 revision Spec-EPUB3 The issue affects the core EPUB 3.3 Recommendation Topic-PackageDoc The issue affects package documents
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants