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

feat(docs): Add an image-border #3147

Merged
merged 6 commits into from
Nov 1, 2024
Merged

feat(docs): Add an image-border #3147

merged 6 commits into from
Nov 1, 2024

Conversation

anna-parker
Copy link
Contributor

@anna-parker anna-parker commented Nov 1, 2024

resolves #

preview URL:

Summary

Adds an outline to images in the docs: as any explanatory images of how our website works include text which is in the same color and format as the docs this can be confusing, here I just add a very simple border around images to make it clearer that they are images:
image

Previously:
image

Screenshot

PR Checklist

  • All necessary documentation has been adapted.
  • The implemented feature is covered by an appropriate test.

@anna-parker anna-parker added the format_me Triggers github_actions to format website code on PR label Nov 1, 2024
@anna-parker anna-parker marked this pull request as ready for review November 1, 2024 08:11
@anna-parker anna-parker changed the title add an image-outline feat(docs): Add an image-outline Nov 1, 2024
@anna-parker anna-parker changed the title feat(docs): Add an image-outline feat(docs): Add an image-border Nov 1, 2024
website/src/styles/mdcontainer.scss Outdated Show resolved Hide resolved
@anna-parker anna-parker merged commit 2849151 into main Nov 1, 2024
19 checks passed
@anna-parker anna-parker deleted the style-images branch November 1, 2024 08:39
@corneliusroemer
Copy link
Contributor

Nice :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
format_me Triggers github_actions to format website code on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants