Thanks to visit codestin.com
Credit goes to github.com

Skip to content

DOC Add horizontal scrolling for wide dataframes - #34155

Merged
glemaitre merged 5 commits into
scikit-learn:mainfrom
jeremiedbb:enable-scrolling-dataframes
Jun 11, 2026
Merged

DOC Add horizontal scrolling for wide dataframes#34155
glemaitre merged 5 commits into
scikit-learn:mainfrom
jeremiedbb:enable-scrolling-dataframes

Conversation

@jeremiedbb

Copy link
Copy Markdown
Member

No description provided.

@DeaMariaLeon DeaMariaLeon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe you want to use the following instead of the selector div.rendered_html (and I think we don't need the max-width).

It works locally. Yeah: locally. :)

.output_subarea.output_html.rendered_html.output_result {
  overflow-x: auto;
}

The thing is, now all wide outputs will be able to scroll. On the other hand, if we only want to target a table, we need to wrap it in a block or something. I think.

I didn't try wrapping the table in a new block or div. I just tried a couple of plots with what I suggest above, and it seems fine.

@DeaMariaLeon

Copy link
Copy Markdown
Member

I forgot to say that it may already work with your selector. But when I first tried with the current doc build, I couldn't see the scroll. I may be missing something.
Feel free to ignore if I just added noise. :)

@jeremiedbb
jeremiedbb marked this pull request as ready for review June 9, 2026 13:48
@DeaMariaLeon

Copy link
Copy Markdown
Member

LGTM, thanks @jeremiedbb

@glemaitre glemaitre left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1

Comment thread doc/scss/custom.scss Outdated
Comment thread doc/scss/custom.scss Outdated
Comment thread doc/scss/custom.scss Outdated
@jeremiedbb

Copy link
Copy Markdown
Member Author

@glemaitre glemaitre left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

@glemaitre
glemaitre merged commit f0dd6f0 into scikit-learn:main Jun 11, 2026
38 checks passed
prady0t pushed a commit to prady0t/scikit-learn that referenced this pull request Sep 2, 2026
@jeremiedbb
jeremiedbb deleted the enable-scrolling-dataframes branch September 3, 2026 15:25
@jeremiedbb jeremiedbb mentioned this pull request Sep 8, 2026
14 tasks
jeremiedbb added a commit that referenced this pull request Sep 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants