Skip to content

Conversation

@gtback
Copy link
Member

@gtback gtback commented Sep 8, 2021

Eventually, we'll want to remove the x-pack role from the Asciidoc for
all sections/headers that include it, but until then this will at least
hide the visual indicator and link. The xpack class will be included
on the surrounding divs, since this is the default Asciidoctor behavior
for "role" tags.

Fixes #2207.

Eventually, we'll want to remove the x-pack role from the Asciidoc for
all sections/headers that include it, but until then this will at least
hide the visual indicator and link. The `xpack` class will be included
on the surrounding divs, since this is the default Asciidoctor behavior
for "role" tags.

Fixes #2207.
@gtback gtback added the docs-build Relates to the build tooling and scripts label Sep 8, 2021
@gtback gtback requested review from nik9000 and olksdr September 8, 2021 19:53
@gtback
Copy link
Member Author

gtback commented Sep 8, 2021

cc: @debadair

@debadair
Copy link
Contributor

debadair commented Sep 8, 2021

Exciting!

Copy link
Contributor

@olksdr olksdr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like there are some lint issues?
otherwise LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-build Relates to the build tooling and scripts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove "x-pack" icon on newer versions of the docs

5 participants