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

Remove the precedence and processing requirements for linked records #2512

Merged
merged 3 commits into from
Jan 9, 2023

Conversation

mattgarrish
Copy link
Member

@mattgarrish mattgarrish commented Jan 6, 2023

As discussed on the 2023-01-05 telecon, this PR removes the precedence rules and processing requirements for linked records.

These rules were added in EPUB 3.1 when there was an expectation that there might be linked browser-friendly json records in the container that reading systems could harvest additional information from. The test results have proven that without that format materializing, support for bibliographic records has not taken off, leaving the precedence and processing rules an unnecessary complexity. This change moves the specification back to its original purpose of allowing linked records for use, for example, with library systems.


Preview | Diff

Comment on lines -878 to -879
<p id="confreq-rs-pkg-link-rendering">Reading systems MUST ignore any instructions contained in
linked resources related to the layout and rendering of the EPUB publication.</p>
Copy link
Member

Choose a reason for hiding this comment

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

This sentence looks a bit different from the previous ones. Was it also discussed that this should go?

(Although... we do not seem to have any test for it anyway, so we may just well remove it...)

Copy link
Member Author

@mattgarrish mattgarrish Jan 6, 2023

Choose a reason for hiding this comment

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

It reads like another case of precedence over what might have been expressed in a browser-friendly package document -- don't use the layout behaviours that were defined for a browser rendering.

You can add properties to the package metadata if you want to control layout and rendering, or create custom attributes in xhtml, so explicitly banning from linked resources when we no longer expect the bff format seems a bit paranoid. No one's going to go through the effort of defining instructions through linked records when the simpler methods exist.

@mattgarrish mattgarrish merged commit e742da4 into main Jan 9, 2023
@mattgarrish mattgarrish deleted the fix/remove-linked-rec-proc branch January 9, 2023 17:17
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.

2 participants