Skip to content

Conversation

hotzenklotz
Copy link
Member

@hotzenklotz hotzenklotz commented Sep 22, 2025

Description:

  • This PR enables image captions for our docs.
  • Updated the pymdown-extensions lib used by mkdocs to v10.12 to enable the captions.
  • Updated the few wk-libs pages, that contain images

Example

Screenshot 2025-09-22 at 10 55 36

Todos:

Make sure to delete unnecessary points or to check all before merging:

  • Updated Changelog
  • Updated Documentation
  • Added / Updated Tests
  • Considered adding this to the Examples

@hotzenklotz hotzenklotz enabled auto-merge (squash) September 22, 2025 09:03
Copy link

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
9327 7808 84% 80% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 5cca033 by action🐍

@hotzenklotz hotzenklotz merged commit 0e4b724 into master Sep 22, 2025
26 checks passed
@hotzenklotz hotzenklotz deleted the docs-image-captions branch September 22, 2025 09:10
hotzenklotz added a commit to scalableminds/webknossos that referenced this pull request Sep 22, 2025
This PR adds image captions to the webknossos documentation. We just
enabled this `mkdocs` features in the
[wk-libs](scalableminds/webknossos-libs#1369)
and this is the follow up here. Most captions were auto-extracted from
the existing images title.

### Example
<img width="1256" height="837" alt="Screenshot 2025-09-22 at 12 01 49"
src="https://github.com/user-attachments/assets/fbf02dba-57b4-49e9-988a-9e435c8325b4"
/>


### Steps to test:
- nothing really

### Issues:
- None

------
(Please delete unneeded items, merge only when none are left open)
- [ ] Added changelog entry (create a `$PR_NUMBER.md` file in
`unreleased_changes` or use `./tools/create-changelog-entry.py`)
- [ ] Added migration guide entry if applicable (edit the same file as
for the changelog)
- [x] Updated [documentation](../blob/master/docs) if applicable
- [ ] Adapted [wk-libs python
client](https://github.com/scalableminds/webknossos-libs/tree/master/webknossos/webknossos/client)
if relevant API parts change
- [ ] Removed dev-only changes like prints and application.conf edits
- [ ] Considered [common edge
cases](../blob/master/.github/common_edge_cases.md)
- [ ] Needs datastore update after deployment

---------

Co-authored-by: Philipp Otto <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants