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

Text overflow in docs #885

Closed
fjanon opened this Issue Jul 14, 2017 · 3 comments

Comments

Projects
None yet
4 participants
@fjanon

fjanon commented Jul 14, 2017

A long description like

http://package.elm-lang.org/packages/elm-lang/core/5.1.1/Platform#program

overflows in the TOC on the right side, apparently because of the CSS

.docs-annotation {
    white-space: pre;

If we remove "white-space: pre;" the text stays in its container.

Should the ".docs-annotation white-space: pre;" be removed?

@process-bot

This comment has been minimized.

Show comment
Hide comment
@process-bot

process-bot Jul 14, 2017

Thanks for the issue! Make sure it satisfies this checklist. My human colleagues will appreciate it!

Here is what to expect next, and if anyone wants to comment, keep these things in mind.

process-bot commented Jul 14, 2017

Thanks for the issue! Make sure it satisfies this checklist. My human colleagues will appreciate it!

Here is what to expect next, and if anyone wants to comment, keep these things in mind.

@jvoigtlaender

This comment has been minimized.

Show comment
Hide comment
@jvoigtlaender

jvoigtlaender Jul 14, 2017

Contributor

This should be an issue in the repo https://github.com/elm-lang/package.elm-lang.org, not of the core library.

Contributor

jvoigtlaender commented Jul 14, 2017

This should be an issue in the repo https://github.com/elm-lang/package.elm-lang.org, not of the core library.

@evancz

This comment has been minimized.

Show comment
Hide comment
@evancz

evancz Jul 14, 2017

Member

Thanks for the report!

It looks like we have elm/package.elm-lang.org#201 as well, so let's go with that one.

Member

evancz commented Jul 14, 2017

Thanks for the report!

It looks like we have elm/package.elm-lang.org#201 as well, so let's go with that one.

@evancz evancz closed this Jul 14, 2017

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