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.

Sign up for GitHub

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

Jump to bottom

Update the XHTML 1.1 RNG schema to latest version #733

Closed
2 tasks done
tofi86 opened this issue Dec 30, 2016 · 4 comments
Closed
2 tasks done

Update the XHTML 1.1 RNG schema to latest version #733

tofi86 opened this issue Dec 30, 2016 · 4 comments
Assignees
@tofi86
Labels
status: has PR The issue is being processed in a pull request type: bug The issue describes a bug
Milestone
4.1.0

Comments

@tofi86
Copy link
Collaborator

tofi86 commented Dec 30, 2016 •
edited
Loading

Currently, as discussed in #696 (comment), the XHTML 1.1 RNG schema in EpubCheck is outdated and/or manually edited.

The RNG schema needs to get updated to the latest XHTML 1.1 version. Also the xhtml11-ent.dtd probably needs an update.

  • update RNG schema
  • update xhtml11-ent.dtd (not needed)
@tofi86 tofi86 added the type: bug The issue describes a bug label Dec 30, 2016
@tofi86 tofi86 added this to the Next milestone Dec 30, 2016
@tofi86 tofi86 mentioned this issue Dec 30, 2016
Closed
@tofi86
Copy link
Collaborator Author

tofi86 commented Dec 30, 2016

Updated the issue description above to reflect changes from comment #696 (comment)

Question: Where are the RNG XHTML 1.1 schemas from? Cannot find an official source?

/cc @rdeltour @mattgarrish

@tofi86 tofi86 changed the title Update the XHTML 1.1 DTD and RNG schema to latest version Update the XHTML 1.1 RNG schema to latest version Dec 30, 2016
@tofi86
Copy link
Collaborator Author

tofi86 commented Dec 30, 2016

Also the xhtml11-ent.dtd probably needs an update.

The entities file is up to date with the latest XHTML 1.1 DTD. Just compared and checked that...

@tofi86 tofi86 self-assigned this Feb 11, 2017
@tofi86 tofi86 mentioned this issue Feb 17, 2017
Merged
@tofi86 tofi86 added the status: has PR The issue is being processed in a pull request label Feb 17, 2017
@tofi86
Copy link
Collaborator Author

tofi86 commented Feb 17, 2017

This will be fixed with PR #747

tofi86 added a commit that referenced this issue Feb 17, 2017
@tofi86
No changes for empty class attributes, added a test anyways // refs #733
2cddb8b
@tofi86
Copy link
Collaborator Author

tofi86 commented Feb 17, 2017

Tasks for Update RNG schema: (https://www.w3.org/TR/xhtml-modularization/Overview.html#status)

  • 1. Changing the datatype of the class attribute so that it permits an empty value
  • Nothing to do here. The class attribute is already defined as <ref name="Text.datatype"/> which allows empty values.
  • Added a test document anyways in 2cddb8b
  • 2. Moving the name attribute for the form and img elements out of the legacy module and into their base modules
  • nothing to do here
  • 3. Changing the datatype for the usemap attribute from IDREF to URIREF

rdeltour pushed a commit that referenced this issue Jun 13, 2017
@tofi86 @rdeltour
Update the XHTML 1.1 RNG schema to latest version
4c6fb49 
* update XHTML 1.1 RNG schema for usemap attribute // refs #696

> Changing the datatype for the usemap attribute from IDREF to URIREF - most user agents require that map references be relative URIs that are local to the document.
 -- see https://www.w3.org/TR/xhtml-modularization/Overview.html#status

* No changes for empty class attributes, added a test anyways // refs #733

Fixes #733
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tofi86 tofi86

Labels
status: has PR The issue is being processed in a pull request type: bug The issue describes a bug
Projects
None yet
Milestone
4.1.0
Development

No branches or pull requests
1 participant
@tofi86