Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
urre
requested review from
luisgoncalves and
markoweb
and
a balanced review from Copilot
October 6, 2026 11:42
urre
commented
Oct 6, 2026
urre
changed the title
urre
changed the title
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The success color fails text contrast requirements, and the code-block dark color branch cannot activate.
Review effort: Balanced
Findings: 2
Open (3)
What changed in this PR
Restyles semantic alerts and updates status colors across the CSS toolkit and documentation.
Changes:
- Adds tinted alerts with mask-based icons and structured content styles.
- Updates semantic colors and button hover states.
- Refreshes documentation styling, examples, and favicon.
| File | Description |
|---|---|
lib/src/curity-theme.css |
Updates semantic status colors. |
lib/src/component/component-alert.css |
Implements the new alert design. |
lib/src/base/base-buttons.css |
Darkens semantic button hover states. |
docs/src/styles/global.css |
Restyles documentation code blocks. |
docs/src/pages/layout/gridlegacy.mdx |
Updates legacy-grid alert markup. |
docs/src/pages/components/alert.mdx |
Documents the redesigned alerts. |
docs/src/components/BaseHead.astro |
Switches to the PNG favicon. |
docs/public/favicon.png |
Provides the new favicon asset. |
.gitignore |
Preserves the source favicon. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| # Grid (legacy) | ||
|
|
||
| <div class="alert alert-warning"><p>This is an older grid system using floats. It is still in the CSS Toolkit but is not recommended to use anymore. Use flexbox or CSS Grid instead.</p></div> | ||
| <div class="alert alert-warning">This is an older grid system using floats. It is still in the CSS Toolkit but is not recommended to use anymore. Use flexbox or CSS Grid instead.</div> |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Restyles alerts as a tinted box with a coloured line icon and updates the status colours to fit the purple scheme (IS-12154).
Related PR:s