Skip to content

Conversation

@ammedd
Copy link
Collaborator

@ammedd ammedd commented Oct 3, 2025

Linking to paper and some older design material

@VeckoTheGecko
Copy link
Collaborator

Fixed @ammedd !

caption: Teaching material
---
ILOs.md
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it possible to add a picture to the tile? It now says "No Thumbnail"

Image

And while the asterisk is nice on the ILOs.md website itself, it may be a bit confusing here? Remove it?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried to add a thumbnail but don't think it worked. Can you have a look why it doesn't work? Is there a better workflow to add thumbnails?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried to add a thumbnail but don't think it worked. Can you have a look why it doesn't work? Is there a better workflow to add thumbnails?

I think the problem here is that this document is a markdown document and not a jupyter notebook - since the package we use for this is nbsphinx its built for Jupyter notebook so adding a markdown file in the gallery is not really supported.

I think that we should just remove the gallery aspect and use a table of contents

diff --git a/docs/conf.py b/docs/conf.py
index c313f6a..ff5c555 100644
--- a/docs/conf.py
+++ b/docs/conf.py
@@ -76,7 +76,6 @@ nbsphinx_thumbnails = {
     "user-guide/assignments/Virtualship_research_proposal": "user-guide/_images/AnnaWeber.jpeg",
     "user-guide/assignments/sciencecommunication_assignment": "user-guide/_images/marine_ss.jpg",
     "user-guide/assignments/Sail_the_ship": "user-guide/_images/vessel.jpg",
-    "user-guide/teacher-content/ILOs": "user-guide/_images/ILOs.jpg",
 }
 
 sphinx_gallery_conf = {"default_thumb_file": "_static/virtual_ship_logo.png"}
diff --git a/docs/user-guide/teacher-content/index.md b/docs/user-guide/teacher-content/index.md
index ba1596b..a561e6a 100644
--- a/docs/user-guide/teacher-content/index.md
+++ b/docs/user-guide/teacher-content/index.md
@@ -16,7 +16,7 @@ The VSC design focuses on creating didactically sound, authentic learning experi
 
 We evaluated in several (under)graduate courses and find that the VirtualShip Classroom is highly engaging, and students report on enhanced confidence and knowledge [(Daniels et al. 2025)](https://current-journal.com/articles/10.5334/cjme.121).
 
-```{nbgallery}
+```{toctree}
 ---
 maxdepth: 1
 caption: Teaching material

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can also update the file to be a jupyter notebook, that's another option

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for checking this out!
Updating the file to notebook sounds good. I'll be filling this section with more content, so it's nice to keep the gallery I guess. I'll think about it more when adding more files.

@ammedd ammedd requested a review from j-atkins October 28, 2025 08:07
@ammedd ammedd merged commit 01dd168 into main Nov 5, 2025
9 of 11 checks passed
@ammedd ammedd deleted the teacher-content branch November 5, 2025 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants