Skip to content

Fix contrast issues with comments in code examples#621

Merged
NickColley merged 1 commit into
mainfrom
fix-comment-colours
Apr 28, 2026
Merged

Fix contrast issues with comments in code examples#621
NickColley merged 1 commit into
mainfrom
fix-comment-colours

Conversation

@NickColley
Copy link
Copy Markdown
Contributor

@NickColley NickColley commented Apr 28, 2026

The colours were updated to better match version 6 but for comments they were mistakenly set to the wrong colour.

This results in comments that are very hard to read.

I have put in a quick patch here to resolve the issue immediately and will upstream this patch into the tech-docs-gem.

Screenshots

Before

Screenshot of code example with comments that have very low contrast

After

Screenshot of code example with comments that have very low contrast

The colours were updated to better match version 6
but for comments they were mistakenly set to the wrong colour.

This results in comments that are very hard to read.

I have put in a quick patch here to resolve the issue immediately and will
upstream this patch into the tech-docs-gem.

alphagov/tech-docs-gem@f09a4b8#diff-81435f36c443c406a3436b0531a4991cd20655917ede750cec10d5b2c6b3bdcf
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 28, 2026

Deploy Preview for govuk-frontend-docs-preview ready!

Name Link
🔨 Latest commit 69ba0ab
🔍 Latest deploy log https://app.netlify.com/projects/govuk-frontend-docs-preview/deploys/69f0db4af0b543000871e080
😎 Deploy Preview https://deploy-preview-621--govuk-frontend-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Contributor

@domoscargin domoscargin left a comment

Choose a reason for hiding this comment

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

Looks good - can we add an issue to remove this fix when we bump tech docs?

@NickColley
Copy link
Copy Markdown
Contributor Author

Will do thank you!

@NickColley NickColley merged commit d53fdb8 into main Apr 28, 2026
5 checks passed
@NickColley NickColley deleted the fix-comment-colours branch April 28, 2026 16:24
@NickColley
Copy link
Copy Markdown
Contributor Author

Ah already raised:

#618

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