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

epub prefix not defined in media overlays #390

Closed
GoogleCodeExporter opened this issue Mar 24, 2015 · 4 comments
Closed

epub prefix not defined in media overlays #390

GoogleCodeExporter opened this issue Mar 24, 2015 · 4 comments
Labels
Topic-MediaOverlays The issue affects media overlays

Comments

@GoogleCodeExporter
Copy link

The epub: prefix is used on attributes in the element definitions without a 
mapping. The only mapping to the ops namespace is given in the informative 
examples.

Either a mapping section is needed, or the mapping should be made explicit at 
the start of the element definitions.

Original issue reported on code.google.com by mgarrish on 15 Sep 2013 at 5:01

@GoogleCodeExporter
Copy link
Author

So I see epub:prefix, epub:type and epub:textref as the attributes with an 
epub: prefix. 

Do you think it would suffice to add a section like "Namespace Prefix Mapping" 
in Content Docs:
http://www.idpf.org/epub/30/spec/epub30-contentdocs.html#sec-overview-nspfx

Or should we also incorporate a "Namespace" heading by the attribute 
definition, such as is seen here:
http://www.idpf.org/epub/30/spec/epub30-contentdocs.html#sec-xhtml-content-type-
attribute


Original comment by marisa.d...@gmail.com on 25 Sep 2013 at 8:55

@GoogleCodeExporter
Copy link
Author

I'd vote to add a namespace prefix mapping section. It'll be easy to adapt
the Content Docs section for MO, and simplifies things if there are other
mappings in the future.

The problem with the attribute definition model is that these aren't
standalone attributes, so we'd have to write the namespace into their text
definitions. I think it's simpler just to add the mapping table.

Original comment by mgarrish on 26 Sep 2013 at 1:52

@GoogleCodeExporter
Copy link
Author

Specification has been updated to add the mappings section:

https://code.google.com/p/epub-revision/source/detail?r=4753

Original comment by mgarrish on 26 Sep 2013 at 9:05

  • Changed state: FinalReview

@GoogleCodeExporter
Copy link
Author

Original comment by marisa.d...@gmail.com on 17 Oct 2013 at 8:12

  • Changed state: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Topic-MediaOverlays The issue affects media overlays
Projects
None yet
Development

No branches or pull requests

1 participant