-
Notifications
You must be signed in to change notification settings - Fork 300
DOCS: Add gallery carousel to docs homepage #6884
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Please see: SciTools/.github#246
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6884 +/- ##
=======================================
Coverage 90.43% 90.43%
=======================================
Files 91 91
Lines 24839 24839
Branches 4656 4656
=======================================
Hits 22463 22463
Misses 1608 1608
Partials 768 768 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
* 'carousel' of github.com:tkknight/iris: Bump the gha group across 1 directory with 3 updates (SciTools#6882)
* upstream/main: Remove flake8 (SciTools#6889)
| } | ||
|
|
||
|
|
||
| def generate_carousel( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
An interesting point raised by @bjlittle. We're using the solution from GeoVista - currently a third-party library. If we got the code from a package like NumPy, there would be some legal hurdles e.g. adding some sort of licence header. @bjlittle has given his blessing, but we probably need to record this in some way. I don't know if there is a legal precedent/convention.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copyright comment added, see 7641299
* 'carousel' of github.com:tkknight/iris: Updated environment lockfiles (SciTools#6887)
🚀 Pull Request
Description
Add a carousel for the gallery on the docs homepage. Note:
conf.pyto create the list ofcardsto use.pydata-sphinx-themeto avoid weird image overlays. See DOCS: gallery carousel images blanking #6885Inspiration for this taken from the GeoVista Docs, credit to @bjlittle for this 🤜 🤛
Screenshot of the carousel (positioned under the main quick button cards on the homepage):
Consult Iris pull request check list
Add any of the below labels to trigger actions on this PR: