✔️ Linting Passed

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

_{Generated for commit: ce2eefe. Link to the linter CI: here}

Last comment, at one point, I think we had the OS automatically chosen based on the browser OS ¹. This would be nice to have too ... I guess this needs some javascript to do this so maybe a different PR.

Sorry for the very late reply.

Maybe try to use substitution to reuse the same conda content in multiple places: https://www.sphinx-doc.org/en/master/usage/restructuredtext/basics.html#substitutions

Not sure if that works with complex multi-line content, if not Including a file may be an option?

I don't think substitutions work for multiline, but yes .. include:: works.

"Packager" sounds a bit weird, maybe "Installer" instead, or "Install method" ? I know it was the wording used previously in scikit-learn 1.4 but still ...

Edit: I have seen that PyTorch uses "Package" which I find slightly weird too ...

Package manager

Changed "Packager" to "Package Manager" as suggested and "OS" to "Operating System" to have a similar length. Long names don't fit well into the same line on small screens so they are only displayed for screen width larger than 960px (I could have played with the flex box to make the captions displayed above the tabs on small screens but I just don't find it necessary).

Last comment, at one point, I think we had the OS automatically chosen based on the browser OS. This would be nice to have too ... I guess this needs some javascript to do this so maybe a different PR.

I did try in my initial rework of the installation page here, but here since we don't have full control of how things are rendered and sphinx-design has those synchonized tabs things etc., it would be harder to correctly override the behavior with JavaScript but I will give it a try.

I did try in my initial rework of the installation page here, but here since we don't have full control of how things are rendered and sphinx-design has those synchonized tabs things etc., it would be harder to correctly override the behavior with JavaScript but I will give it a try.

OK thanks, to be honest, since you have tried already and it does indeed look quite painful, then let's drop this this is not so crucial.

I think the failing tests were unrelated to this PR, probably due to #29100.

Yeah you will need to merge main when #29328 is auto-merged 😓

In it goes, thanks a lot @Charlie-XIAO!

I wished Github has some kind of feature like "merge this in 7 days if noone else comments/complaints", so easy to forget completely fine PRs like this one ...

Oops, looks like we forgot to change a "packager" in the css:

I will provide a quick fix.

Indeed I was looking at the wrong CircleCI artifact sorry for that 😅 ...