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

fix bug in readme display issue #1495 #1724

Merged
merged 1 commit into from
Feb 11, 2024
Merged

Conversation

ValerasNarbutas
Copy link
Contributor

Q A
Bug fix? [x]
New feature? [ ]
New sample? [ ]
Related issues? fixes ##1495

What's in this Pull Request?

Having sh incode does not compile in web version, changed to bash

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you for the first contribution to this project.

@michaelmaillot michaelmaillot self-assigned this Feb 10, 2024
@michaelmaillot
Copy link
Collaborator

michaelmaillot commented Feb 10, 2024

Hi @ValerasNarbutas,

Thanks for this PR!

May I ask you which version of Material for MkDocs did you use during your tests?

Because even if there's no doubt that we have to upgrade current supported version (3.1.0), I've tested your changes with this one and the display is still wrong.

But if I remove tabs indent for the code blocks, it's working. Example with the contributing.md page:

Before

image

image

After

image

image

I thinkg that for now, the best fix would be to remove tabs in the code blocks. Then we'll update the version of Material for MkDocs.

Are you ok with this?

@ValerasNarbutas
Copy link
Contributor Author

Yes, off course!

Not sure about version:) had similar issue in my repo, now thinking maybe I need to recheck it with your solution.

@AJIXuMuK AJIXuMuK merged commit d32a3c5 into pnp:dev Feb 11, 2024
1 check passed
@AJIXuMuK
Copy link
Collaborator

Thanks you @ValerasNarbutas!

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