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

Use font awesome anchor #789

Merged
merged 1 commit into from
Dec 6, 2021
Merged

Conversation

jkremser
Copy link
Member

@jkremser jkremser commented Dec 6, 2021

follow-up of #692 (comment)

looks like this:

Screenshot 2021-12-06 at 15 12 33

Signed-off-by: Jirka Kremser jiri.kremser@gmail.com

@jkremser jkremser closed this Dec 6, 2021
@jkremser jkremser reopened this Dec 6, 2021
@jkremser jkremser closed this Dec 6, 2021
@jkremser jkremser reopened this Dec 6, 2021
@netlify
Copy link

netlify bot commented Dec 6, 2021

✔️ Deploy Preview for k8gb-preview ready!

🔨 Explore the source changes: 3993475

🔍 Inspect the deploy log: https://app.netlify.com/sites/k8gb-preview/deploys/61ae2b2e395b7e00088b791a

😎 Browse the preview: https://deploy-preview-789--k8gb-preview.netlify.app

@@ -45,7 +46,7 @@ <h2 class="project-tagline">{{ page.description | default: site.description | de
</header>

<main id="content" class="main-content" role="main">
{% include anchor_headings.html html=content anchorBody="§" anchorClass="heading-anchor" anchorAttrs="aria-labelledby=\"%html_id%\"" headerAttrs="class=heading" %}
{% include anchor_headings.html html=content anchorClass="heading-anchor fa fa-anchor" anchorAttrs="aria-labelledby=\"%html_id%\"" headerAttrs="class=heading" %}
Copy link
Contributor

Choose a reason for hiding this comment

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

I suggest to use fa-link symbol
image

instead of fa-anchor
image

Most documentation-centric themes and CMS are using "link" symbol for header anchors and it became a de-facto standard recognizable by users.

Copy link
Member Author

Choose a reason for hiding this comment

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

well, both are better than §. I like the "anchor" anchor better b/c it's aligned w/ the k8s nomenclature but I guess we can be boring predictable in our docs ;) so changing..

@somaritane somaritane self-requested a review December 6, 2021 15:16
Copy link
Contributor

@somaritane somaritane left a comment

Choose a reason for hiding this comment

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

@jkremser looks awesome, just one nit about symbol

Signed-off-by: Jirka Kremser <jiri.kremser@gmail.com>
@jkremser jkremser merged commit 2297452 into k8gb-io:gh-pages Dec 6, 2021
@jkremser jkremser deleted the gh-pages-anchor branch December 6, 2021 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants