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

Add enum value separators #1572

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dyladan
Copy link
Member

@dyladan dyladan commented Nov 12, 2024

Add separators in order to separate enum value tables from the footnotes above.

Fixes #1569

Add separators in order to separate enum value tables from the footnotes above.

Fixes open-telemetry#1569
@jsuereth
Copy link
Contributor

@dyladan I really like this. Is this in DRAFT for discussion or other reasons?

@dyladan
Copy link
Member Author

dyladan commented Nov 13, 2024

There are two versions of it if you look at the linked issue. I didn't want to go through the hassle of keeping two PR's up-to-date before we decided which one we would rather use.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

[rendering] Enum values not sufficiently separate from attribute footnotes
2 participants