Skip to content

add cvss badge to riskhandling, because it confused users that cvss a…#767

Merged
timbastin merged 1 commit into
mainfrom
rework-risk-badge
May 28, 2026
Merged

add cvss badge to riskhandling, because it confused users that cvss a…#767
timbastin merged 1 commit into
mainfrom
rework-risk-badge

Conversation

@5byuri

@5byuri 5byuri commented May 28, 2026

Copy link
Copy Markdown
Member

…re not highlighted

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR makes CVSS values in the risk-handling UI visually consistent with adjacent risk badges by reusing the shared CVSSBadge component and updating that badge to include a severity label.

Changes:

  • Adds a local CvssCell helper for consistent CVSS/N/A rendering in risk-handling rows.
  • Replaces plain CVSS text in package, CVE, and vulnerability rows with CVSSBadge.
  • Updates CVSSBadge to render SEVERITY (score) instead of only the score.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/components/risk-handling/RiskHandlingRow.tsx Uses CVSSBadge consistently across dependency risk table CVSS cells.
src/components/common/Severity.tsx Updates shared CVSS badge text to include the severity label.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@timbastin timbastin merged commit 996a6ff into main May 28, 2026
17 of 19 checks passed
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.

3 participants