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

fix: Remove emphasis from xref in headings and fix xrefs in headings #1029

Merged
merged 4 commits into from
Dec 18, 2023

Conversation

kesara
Copy link
Member

@kesara kesara commented Sep 4, 2023

  • This change removes emphasis on xref content in headings.
  • This change also adds xref content to name when the format="none" in the xref element.
  • When xref is in title, if xref has a target to a reference use
    square brackets ([ <target text> ]).
  • If the xref target is not a reference don't use brackets or parenthesis.
  • If the xref element has text content:
    For reference targets:
    <text content> [<target>]
    For other targets:
    <text content> (<target>)

Fixes #683

@kesara kesara changed the title fix: Add xref content to name when format is none fix: Remove emphasis from xref in headings and fix xrefs in headings Sep 6, 2023
@kesara kesara marked this pull request as ready for review September 14, 2023 03:49
@kesara kesara added the rpat Issues needing attention from RFC Production Advisory Team label Sep 14, 2023
@kesara
Copy link
Member Author

kesara commented Sep 14, 2023

See #683 (comment) for examples.

This change removes emphasis on xref content in headings.
This change also adds xref content to name when the format="none" in the xref element.

Fixes ietf-tools#683
When `xref` is in `title`, if `xref` has a target to a `reference` use
square brackets (`[ <target text> ]`).
If the `xref` target is not a reference don't use brackets or parenthesis.

If `xref` element has text content:
For `reference` targets:
    `<text content> [<target>]`
For other targets:
    `<text content> (<target>)`
@kesara kesara merged commit e067417 into ietf-tools:main Dec 18, 2023
13 checks passed
@kesara kesara deleted the fix/name-xref branch December 18, 2023 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rpat Issues needing attention from RFC Production Advisory Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

xref in section title (< name >) broken in HTML
2 participants