-
Notifications
You must be signed in to change notification settings - Fork 41
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
<iref> in <table> not supported #378
Comments
@henrik@levkowetz.com commented This is intentional. Could be wrong, of course. The reason is as given here: https://zin.tools.ietf.org/html/draft-levkowetz-xml2rfc-v3-implementation-notes-06#section-3.1.7 |
@julian.reschke@gmx.de commented I agree that there's a spec problem. However, that needs to be fixed on them HTML level. FWIW, RFC 7992 tries to specify all kinds of things without having to, and this leads to problems like these. rfc2629.xslt generates:
So the table is enclosed in a div anyway, and that allows div child elements that can be used as targets for the references. There are more ways to do this, though, and I believe RFC 7992 shouldn't mandate a specific way to do it. |
@henrik@levkowetz.com commented Yes, in many places in the html renderer I use enclosing divs in order to have something for an ID to be attached to. In some cases, such as link targets for caption text links, I use spans inserted immediately in front of a table or figure. In this case, the idiocy of the specification and the extent of the special-casing needed in the code made me choose a different path. Doing something else isn't hard, it's just a bother to work out what to do when the specification leads to invalid html. Is there any fundamental reason to prefer enclosing divs as opposed to sibling spans when in an inline rendering context? |
@henrik@levkowetz.com changed _comment0 which not transferred by tractive |
@julian.reschke@gmx.de commented It's really a matter of taste. I like the enclosing model when it's used for the user-specified anchor. |
@henrik@levkowetz.com commented Ok, thanks Julian. |
@henrik@levkowetz.com changed status from |
@henrik@levkowetz.com changed resolution from `` to |
@henrik@levkowetz.com commented Fixed in ecb73cb: Added back the ability to place elements in locations where they will translate to invalid HTML. Avoided invalid HTML by pushing the span up one level, as a previous sibling, when needed. Fixes issue #378. |
resolution_fixed
type_defect
| by julian.reschke@gmx.deThe following construct:
fails with:
Error: Did not expect element iref there, at /rfc/middle/section581394b850b8186c0bd36f7dddcae5d3ba5371e4/section5f1f8b9ced72dea365b98717219bbeadc86cce74/table/iref
Issue migrated from trac:378 at 2022-02-05 12:46:36 +0000
The text was updated successfully, but these errors were encountered: