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.

Sign up for GitHub

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

Jump to bottom

docs(python): Don't break lines in nav but let it scroll #5341

Closed
wants to merge 1 commit into from
Closed

docs(python): Don't break lines in nav but let it scroll #5341

wants to merge 1 commit into from

Conversation

vvilhonen
Copy link

Change

Screenshot 2022-10-26 at 19 09 22

into

Screenshot 2022-10-26 at 19 12 24

@github-actions github-actions bot added docs python Related to Python Polars labels Oct 26, 2022
@ghuls
Copy link
Collaborator

ghuls commented Oct 26, 2022

This was added on purpose, see:
#4604

@vvilhonen
Copy link
Author

vvilhonen commented Oct 27, 2022 •
edited
Loading

Scrolling seems less bad trade off to me than seeing half of the function name like ’ff’, ’val’, ’rst’.

@alexander-beedie
Copy link
Collaborator

alexander-beedie commented Oct 29, 2022 •
edited
Loading

FYI: I finally solved this issue for good - PR coming shortly... 🎉

eg: polars.internals.series.string.StringNameSpace.to_lowercase >> polars.Series.str.to_lowercase

@alexander-beedie alexander-beedie mentioned this pull request Oct 29, 2022
Merged
@vvilhonen
Copy link
Author

Thanks, that's way better!

@vvilhonen vvilhonen deleted the docs/dont-break-lines-in-nav branch October 30, 2022 07:11
@stinodego stinodego added the documentation Improvements or additions to documentation label Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
documentation Improvements or additions to documentation python Related to Python Polars
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@vvilhonen @ghuls @alexander-beedie @stinodego