Skip to content

Fix Snyk badge, add ti&m logo - #1371

Merged
rickbrouwer merged 2 commits into
gitlabform:mainfrom
marcelstoer:fix/docs
Jul 21, 2026
Merged

rickbrouwer merged 2 commits into
gitlabform:mainfrom
marcelstoer:fix/docs

Conversation

@marcelstoer

@marcelstoer marcelstoer commented Jul 21, 2026 •

Copy link
Copy Markdown
Contributor

2 commits with 1 fix each.

@rickbrouwer

Copy link
Copy Markdown
Collaborator

Hi!

Thanks for the PR. The logo doesn't align well with the rest. Do you think this can be improved?

image

@marcelstoer

marcelstoer commented Jul 21, 2026 •

Copy link
Copy Markdown
Contributor Author

Odd, it renders fine with my Markdown editors and in the generated MkDocs site. The new entry uses the same styles as the existing ones.

Screenshot 2026-07-21 at 14 03 49 Screenshot 2026-07-21 at 14 04 58 Screenshot 2026-07-21 at 14 05 11

@marcelstoer

Copy link
Copy Markdown
Contributor Author

It's fine even if you reduce the viewport width.

Screenshot 2026-07-21 at 14 09 57

However, I noticed that the GitHub Markdown renderer puts all logos on a single line. Then things start looking obvious.

Screenshot 2026-07-21 at 14 10 14

The new logo image is square but the logo isn't. I'll crop it properly.

@rickbrouwer rickbrouwer left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks!

@rickbrouwer
rickbrouwer enabled auto-merge (squash) July 21, 2026 14:09
@rickbrouwer
rickbrouwer disabled auto-merge July 21, 2026 14:09
@rickbrouwer
rickbrouwer merged commit 31fa34a into gitlabform:main Jul 21, 2026
21 of 23 checks passed
@marcelstoer
marcelstoer deleted the fix/docs branch July 21, 2026 14:36

This branch had an error being deployed

1 failed deployment
Integrate Pull Request — cccb308e Deployed Jul 21, 2026 by marcelstoer via Acceptance Tests / GitLab Ultimate #134
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.

2 participants