Skip to content

Fix docs site title contrast#90

Merged
dbpolito merged 4 commits intomainfrom
fix/site-title-color
Apr 3, 2026
Merged

Fix docs site title contrast#90
dbpolito merged 4 commits intomainfrom
fix/site-title-color

Conversation

@dbpolito
Copy link
Copy Markdown
Contributor

@dbpolito dbpolito commented Apr 3, 2026

Reply to addressed PR review threads

@dbpolito dbpolito self-assigned this Apr 3, 2026
Copy link
Copy Markdown

@kompassdev kompassdev bot left a comment

Choose a reason for hiding this comment

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

★★☆☆☆

The CSS change for site title contrast looks good, but the MDX file contains debug text that appears to be unintended.

packages/web/src/content/docs/docs/index.mdx:6 - Added debug text "qwewqewq." to the introduction paragraph that should not be committed to production documentation.

Copy link
Copy Markdown

@kompassdev kompassdev bot left a comment

Choose a reason for hiding this comment

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

★★☆☆☆

While the CSS change for site title contrast is correct, the MDX file still contains debug text that must be removed before merging.

Note: In addition to the already-flagged "qwewqewq." on line 6, there's a second instance of debug text "asdsadsa." on line 14 that also needs to be removed.

packages/web/src/content/docs/docs/index.mdx:14 - Added debug text "asdsadsa." to the first bullet point in the "What Kompass is" section that should not be committed.

@dbpolito dbpolito merged commit 70d14b1 into main Apr 3, 2026
1 check passed
@dbpolito dbpolito deleted the fix/site-title-color branch April 3, 2026 15:25
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.

1 participant