Skip to content

fix(#1866): restore bootstrap grid classes to fix huge images#2211

Open
ayushman1210 wants to merge 2 commits into
medic:mainfrom
ayushman1210:1866
Open

fix(#1866): restore bootstrap grid classes to fix huge images#2211
ayushman1210 wants to merge 2 commits into
medic:mainfrom
ayushman1210:1866

Conversation

@ayushman1210
Copy link
Copy Markdown
Contributor

Description

Fixes #1866.

Some images on the documentation site were rendering at their original, huge sizes because the new theme does not support the legacy Bootstrap grid classes (col-6, col-lg-8, right, etc.) that are extensively used in the figure shortcodes across the markdown files.

This PR adds these legacy utility classes to assets/css/custom.css. By doing so, it restores the original layout constraints and correctly scales down the images across the entire documentation site, without requiring manual updates to hundreds of markdown files.

@mrjones-plip mrjones-plip self-requested a review May 6, 2026 13:12
Copy link
Copy Markdown
Contributor

@mrjones-plip mrjones-plip left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @ayushman1210 !

There should be no CSS removed in this PR, correct? If yes, please explain why, otherwise please restore the CSS that was removed, including comments.

Please add a comment as to why the old CSS is being added back in.

Finally, please add three examples of before and after screen shots showing the images being too big and then being fixed with the new CSS. Thanks!

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.

Review image size with new theme

2 participants