-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Fix callout numbering #22794
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 callout numbering #22794
Conversation
|
The preview will be available shortly at: |
26ca528 to
855b430
Compare
|
@bobfuru, there is this fix: Didn't realize it was possible, but an SME had it in a contributed PR. |
|
@bobfuru, would love to learn of a clean solution, though. |
@jboxman I've used this before but couldn't get it to work in this case, which has an admonition note within two callout descriptions. It's also inside of a numbered step, though not sure that should matter. I ended up just making the note part of <1>. By coincidence, I think it improved readability. |
|
/cherrypick enterprise-4.5 |
|
/cherrypick enterprise-4.4 |
|
@bobfuru: new pull request created: #22795 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
@bobfuru: new pull request created: #22796 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Relates to this PR 22785 and this comment:
#22785 (comment)
@bmcelvee Unfortunately, when I add the continuing
+, it doesn't continue the callout numbering. (Instead, it lists both callouts as #1.) I couldn't figure out a better workaround, and ultimately reverted back to original formatting. I actually think I mucked with this when the original content was created. Will continue to investigate a fix for this.