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

Inline code literals have no wrapping #321

Closed
LivInTheLookingGlass opened this issue Aug 30, 2016 · 3 comments · Fixed by #1050
Closed

Inline code literals have no wrapping #321

LivInTheLookingGlass opened this issue Aug 30, 2016 · 3 comments · Fixed by #1050
Labels
Bug A bug Needed: design decision A core team decision is required

Comments

@LivInTheLookingGlass
Copy link

I'm trying to give examples of how an object is serialized. I need this to wrap properly.

When I use the default style, it looks like this:

If I use the solution in mkdocs/#709, it looks like:

The first and last literal look fine, but the middle one is insanity. Can you help me fix/explain what's going on here? And if we arrive at a satisfactory solution, could this be integrated into the default theme?

@Blendify
Copy link
Member

Blendify commented Mar 9, 2017

What do we want to do here? Add a scroll bar or wrap the text?

@Blendify Blendify added Needed: design decision A core team decision is required Needed: patch A pull request is required labels Mar 9, 2017
@jessetan
Copy link
Contributor

I prefer scrolling. Opnions?

@stsewd
Copy link
Member

stsewd commented Feb 10, 2021

+1 on wrapping, adding a scrollbar would make it more "big" or look weird being inside a paragraph

stsewd added a commit that referenced this issue Feb 10, 2021
This preservers white spaces as is when copying,
so all good.

Fixes #321
stsewd added a commit that referenced this issue Feb 23, 2021
This preservers white spaces as is when copying,
so all good.

Fixes #321
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A bug Needed: design decision A core team decision is required
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants