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

Skip to content

docs: implement responsive multi-column layout for emeritus contributors to reduce whitespace #31565

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

Merged
merged 2 commits into from
Jun 18, 2025

Conversation

bl3ck
Copy link
Contributor

@bl3ck bl3ck commented Jun 17, 2025

Reference Issues/PRs

References #31548

What does this implement/fix? Explain your changes.

This PR improves the layout of the emeritus contributors section on the About Us page by introducing a new utility class: grid-list-three-columns, which provides a responsive, three-column layout.

The updated layout reduces excess whitespace and helps users reach key sections more quickly, as recommended by @reshamas. The layout switches to a single column on smaller screens for better accessibility and readability.

Results:

desktop

mobile

Any other comments?

This change also helps maintain visual consistency with other parts of the documentation that use multi-column formatting.

Copy link

github-actions bot commented Jun 17, 2025

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: abf9e92. Link to the linter CI: here

@bl3ck bl3ck changed the title docs: implement responsive multi-column layout for emeritus contributors to reduce whitespace (#31548) docs: implement responsive multi-column layout for emeritus contributors to reduce whitespace Jun 17, 2025
@reshamas
Copy link
Member

@bl3ck Thank you for the PR. The layout looks good, and doesn't require manual insertion of names and we can keep the maintainers_emeritus.rst file, so that's a plus.

I am not sure why the tests are failing.

@bl3ck
Copy link
Contributor Author

bl3ck commented Jun 17, 2025

Hi @reshamas, happy to contribute! Your work with scikit-learn and open source is a big reason I wanted to get involved.

The test failure looks like an intermittent issue with fetch_california_housing—Figshare sometimes returns a 403 Forbidden from CI environments, so it's likely not PR-related. I’ll look into it further and also try re-running the test.

@bl3ck
Copy link
Contributor Author

bl3ck commented Jun 17, 2025

@reshamas All tests pass now!

Copy link
Member

@adrinjalali adrinjalali 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 so much @bl3ck

@adrinjalali adrinjalali merged commit 0fb2355 into scikit-learn:main Jun 18, 2025
36 checks passed
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