Agreed. At the very least, adding a parent div with a new class is a good start. If CSS flex can be added on top in a way that can be overridden/disabled easily in downstream CSS, then that sounds good, too. If it can't easily be disabled, we could still consider adding it -- but the chances of breaking user sites (in a way that's hard to fix) goes up, so we'd have to think harder about the tradeoffs.

... regarding a new dependency, maybe that could be opt-in, like pip install sphinx-gallery[design], and then people can do in conf.py:

sphinx_gallery_conf = {
    ...
    'use_sphinx_design': True,
}

or something. But I am no expert here, happy to hear what you (and others) think @alexisthual !

I tried something locally:

• adding the new div is easy and does most of the job
• designing a responsive grid also works great, appart from the fact that some unexpected divs appear in the DOM due to sphinx; we'll need a fix for this issue, otherwise we can't use grids

My take is that we don't need sphinx-design in the end.

However, I'd be in favour of doing a small refactoring the thumbnail. Currently, the div's height won't scale to it's content's height. I guess this is because html figure captions don't play well with divs. It seems that a min-height is set to fix this, but it's not the right way of doing this.

The following structure would make sense to me:

<a class="sphx-glr-thumbcontainer">
  <img />
  <div class="sphx-glr-thumb-title">title</div>
</a>

as opposed to the current structure:

<div class="sphx-glr-thumbcontainer">
  <figure class="align-default">
    <img />
    <figcaption>
      <p><span class="caption-text"><a class="reference internal"><span class="std std-ref">title</span></a> 
      </span><a class="headerlink" title="Permalink to this image">¶</a></p>
    </figcaption>
  </figure>
</div>

I tested it and it feels much nicer. This could potentially break some css added by users on top of sphinx-gallery, but I think they added this code mostly to fix the issue mentioned above.

What do you think? 🙂

I tested it and it feels much nicer. This could potentially break some css added by users on top of sphinx-gallery, but I think they added this code mostly to fix the issue mentioned above.

I agree that this is most likely the case. If we can git image + caption to fit without any trickery, and make some clear note about how to adjust CSS to deal with the change, I think it can be considered a bugfix and released without a deprecation cycle.