Skip to content
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

center images and captions #886

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

center images and captions #886

wants to merge 2 commits into from

Conversation

Aprillion
Copy link
Collaborator

@Aprillion Aprillion changed the title center images and captions #766 center images and captions Feb 15, 2025
@LeMurphant
Copy link
Collaborator

This works as expected, but centering is often not pretty. I will keep the mechanism of "apply formatting to after image" and attempt to find a nicer way to format it before merging.

@LeMurphant
Copy link
Collaborator

After some feedback from the team, we decided to use a Caption: mechanism instead
caption

@Aprillion
Copy link
Collaborator Author

Aprillion commented Feb 20, 2025

https://stampy-ui2.aprillion.workers.dev/ is updated to latest commit .. I have no idea which document uses the new caption to test, but updated code looks good to me

@LeMurphant
Copy link
Collaborator

@Aprillion you can see the result on these pages
https://stampy-ui2.aprillion.workers.dev/questions/8241/What-is-interpretability-and-what-approaches-are-there
https://stampy-ui2.aprillion.workers.dev/questions/0
The former has an issue with the second image, but it seems unrelated to the PR, it has the same issue on the live site

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.

No way to center image captions
2 participants