✔️ Linting Passed

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

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

Example here

TODO:

1. Address Olivier's comment (during a call). He mentioned that the ReprHTMLMixin could be set-up in a way that we only add documentation link of estimators that have similar HTML structure.
2. Test for broken links
3. The "?" link is too small, but it's the same size of the text. latest doc built - It seems that having the same text for each hyperlink is different than the recommended "best practices".

tested on the example you linked and it worked fine except for the C parameter of LogisticRegression. Maybe the text fragment requires more than 1 character ?

Thanks for the feedback! - the link to C that you mention works for me locally. But the link to my example doesn't work (the whole example, not the parameter link). Maybe on the next doc build I can see it.. ?
I addressed some of the comments you left. I'll work on the rest.

Docs build failure unrelated. See #31918 (comment)

I would like to know if I should implement:

If we go for the tooltip, then I don't think that a link for each parameter is needed anymore. The link for the estimator is enough then. And the tooltip could be displayed on hovering over the param name cell.

Up to now, it looks like below. Example here

After changes requested by Guillaume
example

I think that we need to keep the <br> because with bullet point list, we are going to join each line which is not a nice rendering.

Since we are writing a 88 characters length line, I think that we should be OK with the length of the original docstring.

It could be nice to interpret the RST to HTML but it would require additional dependencies and it will be rather annoying.

So, Guillaume's previous comment overrides Olivier's comment here. I added back the <br>

There is now a scroll inside the tooltip (not obvious in the screen-shoot).
I fixed a bunch of issues with the latests changes, so I'm waiting for feedback before working again on the tests.

I like what I see in the screen shot. Do you mind pushing a new commit with a doc build to try it out live (@glemaitre cancelled it by sync'ing with main) ?

The documentation build is available @jeremiedbb

I find this new iteration really nice and better than the previous ? cell. I'm not sure that linking to the full doc page is necessary then but why not if you want to have the rst properly rendered.

One thing that is not very easy is to enable the focus to be able to scroll within the tooltip (I have a laptop with a mouse pad). When I hover on a param name, the tooltip appears and then if I move my cursor to enter the tooltip and scroll, it just displays the tooltip of the next param instead. I have to hover and move quickly to be able to enter the tooltip with my cursor.

I noticed that the scroll bar doesn't appear when the tooltip pops up on hovering the param name. It appears only when I managed to make my cursor enter the tooltip. I believe that if it appeared right away I might be able to scroll without having to move the cursor inside the tooltip, not sure though.

One thing that is not very easy is to enable the focus to be able to scroll within the tooltip (I have a laptop with a mouse pad).

Yep it is a bug. We need to figure out the right way to do so.