Fix padding for anchor-linked workflow images

[lochhh]

In Docfx images are made clickable when the size exceeds 200 x 200.

This PR closes #19 by adding a selector for workflow <img> elements that are direct children of <a> tags.

The alternative is to apply the CSS style to all <img> elements in workflow:

.workflow img {
  padding-top: 12px;
  padding-left: 12px;
  padding-right: 12px;
}

[CLAassistant]

All committers have signed the CLA.

[banchan86]

Thank you, @lochhh, for investigating the issue and providing a fix—I’ll have to try it out! Do you know why the clickable script causes those images to lose their padding?

[lochhh]

Thank you, @lochhh, for investigating the issue and providing a fix—I’ll have to try it out! Do you know why the clickable script causes those images to lose their padding?

The script adds the anchor as a parent of the img, so the existing css selector (.workflow > p > img) no longer applies:

<div class="workflow hljs">
  <p>
    <a target="_blank" rel="noopener noreferrer nofollow" href="https://bonsai-rx.org/docs/workflows/bonvision-render-randomgrating.svg">
      <img src="https://bonsai-rx.org/docs/workflows/bonvision-render-randomgrating.svg" alt="BonVision render random grating">
    </a>
  </p>
</div>

c.f. smaller images:

<div class="workflow hljs">
  <p>
    <img src="https://bonsai-rx.org/docs/workflows/bonvision-render-1grating.svg" alt="BonVision render grating">
  </p>
</div>

[banchan86]

this issue has been closed but just wanted to add a note based on a discussion @open-ephys/bonsai-onix1-docs#181 that we might want to investigate disabling the click functionality which seems to have been added for image gallery functionality ( for instance, https://dotnet.github.io/docfx/extensions/templates.html). I still need to see if workflows with images >200px are clickable with this fix.

• I tried looking at docfx.json, but it doesn't seem to be an exposed property.
• As the script is part of the docfx core modern template (specifically the src/markdown.ts file), I tried to see if it was possible to overwrite it with a copy of the file (similar to how Cris did it for the managedreference templates) but seems like only those files which can be seen by docfx template export modern can be overwritten.