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

docs: Fix compatibility issues with opentelemetry.io #953

Merged
merged 1 commit into from
Sep 24, 2021
Merged

docs: Fix compatibility issues with opentelemetry.io #953

merged 1 commit into from
Sep 24, 2021

Conversation

arielvalentin
Copy link
Contributor

The changes I submitted in #930 were optimized for a GitHub markdown, however when I integrated them into opentelemetry.io I ran into issues with navigation, duplicate headers, and some odd rendering issues.

The changes included in this PR update the Markdown to work better with Hugo and opentelemetry.io:

2021-09-22_13-35-01

2021-09-22_13-40-09

Copy link
Contributor

@ahayworth ahayworth left a comment

Choose a reason for hiding this comment

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

I have not the tiniest clue how these changes actually fix the visual problems, but I absolutely trust that they do. 😄 😄 😄 😄

@arielvalentin
Copy link
Contributor Author

@ahayworth The main problem was I reused the same description on every page with an image of a Ruby. I thought it was a "meta" description but no it was included as a header on every page.

@ahayworth
Copy link
Contributor

@ahayworth The main problem was I reused the same description on every page with an image of a Ruby. I thought it was a "meta" description but no it was included as a header on every page.

Ah that makes sense!

@arielvalentin
Copy link
Contributor Author

Oh also navigation is busted because the io site processes the MD to generate static HTML. I had to sacrifice MD navigation as a result.

Copy link
Contributor

@fbogsany fbogsany left a comment

Choose a reason for hiding this comment

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

It's fine to break MD navigation - these files are intended purely for opentelemetry.io.

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.

3 participants