Closed prjemian closed 2 months ago
Might get a lot easier if we add the sphinx-design package.
(That's what bluesky-adaptive is using)
Half the lines of code, no need for raw html, easier to manage.
Download a preview copy of the web site for review from docs actions page. The download is at/near the bottom of the page.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Totals | |
---|---|
Change from base Build 8543931001: | 0.0% |
Covered Lines: | 983 |
Relevant Lines: | 1189 |
I actually find this a lot better - much less overwhelming. I can tell exactly where I want to go. I don't really see any value in the preview of "some" of the content. I'd rather just pick a category, and then browse from there.
View of checklist now (text is pulled from the section title by Sphinx):
One more step, at least, to modularize the content after the installation checklist. See TODO
items, such as https://github.com/BCDA-APS/bluesky_training/blob/dc30425bc9ce92a221f2d08e3b82e3bbcb1f74f5/docs/source/instrument/install_new_instrument.rst?plain=1#L22
Published latest version to public website.
Ready for review.
@MDecarabas -- re-examine the checklist: https://bcda-aps.github.io/bluesky_training/instrument/install_new_instrument.html#instrument-install
Thanks MDecarabas!
Looks great!