Add new section for custom fxl properties#2138
Conversation
murata2makoto
left a comment
There was a problem hiding this comment.
If I am not mistaken, custom attributes are defined in EPUB 3.3 and then described in EPUB 3.3 RS, but custom fxl properties are described in EPUB 3.3 RS only. I think that we should be consistent.
Yes, but this is only because it would otherwise be invalid to have custom attributes in xhtml content documents -- although allowed by xml, they aren't an officially recognized extensibility mechanism, so you have to extend html to allow other attributes. We don't need to do anything special to allow authors to use custom properties in the package document as it's no different than using any other metadata vocabulary. You just have to define a prefix. |
|
Sorry. I think that almost everything in 6.1.5 Custom Properties of EPUB 3.3 RS and 4.1.1.7 Custom Attributes is irrelevant to the conformance requirements of EPUB 3.3 RSes. But we might want to have something like "Reading Systems MAY behave differently due to custom attributes, as long as they satisfy the requirements in this specification." in 4.1.1.7 of EPUB 3.3 RS. We might want to have something similar about custom properties in EPUB 3.3 RS. 6.1.5 and 4.1.1.7 have four sentences beginning with "Vendors may". I think that these sentences should be reformulated as requirements on data and should then be moved to EPUB 3.3. In the case of custom properties, probably in D.1.6. |
restructure RS spec to match new layout rendering section in core
|
I've tried to rework the requirements in the latest commit so that they can exist on the authoring side. I've also restructured the RS spec to match the new layout control section of the core spec (i.e., grouped fxl and reflowable properties). Nothing has changed in those sections; only the custom properties section that now follows them is new to this PR. |
|
Thanks. I'm still proofreading it. It appears that "diff" screwed up. ;-( |
The reading system diff/preview links had a typo in the branch name. I've fixed them in the first comment, so they should work now. (Oh, and I see @iherman already caught the problem yesterday in his comment.) |
|
In the section "Custom attributes" in EPUB 3.3 Core,
I would like to reformulate these paragraphs as requirements on XHTML content documents. After all, we cannot create a test for testing developers. Here is my proposed rewrite.
I do not think that we should mention the purposes of custom attributes. If we do, it will become unclear whether or not custom attributes NOT FOR RENDERING are allowed.
I was heavily involved in the design of XML namespaces, but the term "default namespace" continues to cause headaches to me. Do we lose something by dropping this sentence? I am not sure if this sentence correct. |
Well, I was going to argue that we're moving away from an extension mechanism solely for RS developers, which is what this was only supposed to be, to a mechanism that can be exploited for any purpose, but it seems epubcheck just blanket allows any attributes in foreign namespaces...
Can we work around it by saying:
(edit: reworded slightly from original because "provided" left the definition of "custom" unclear.) |
Yes, this rewrite works.
An earlier version of ISO/IEC 29500-2 (OOXML) contained lots of sentences beginning with "Producers shall", "Consumers may", "Format designers shall", and so forth. I rewrote most of them in the latest version as requirements on data. I firmly believe that data conformance, which is enforceable by examining data, is far better than application conformance, which is hard to test. |
updated change log entries
I've largely copied the "custom attributes" section, but made more clear here that custom properties should only address reading system-specific rendering issues.
Fixes #2134
Preview | Diff