Skip to content

Added CVSS vector handling and calculator integration for vulnerability impact details#667

Open
stijnpotters1 wants to merge 3 commits into
masterfrom
feat/calculate-ff-impact
Open

Added CVSS vector handling and calculator integration for vulnerability impact details#667
stijnpotters1 wants to merge 3 commits into
masterfrom
feat/calculate-ff-impact

Conversation

@stijnpotters1

@stijnpotters1 stijnpotters1 commented Jul 21, 2026

Copy link
Copy Markdown
Contributor
image

@stijnpotters1 stijnpotters1 self-assigned this Jul 21, 2026
@stijnpotters1 stijnpotters1 linked an issue Jul 21, 2026 that may be closed by this pull request
@stijnpotters1
stijnpotters1 requested a review from Matthbo July 21, 2026 09:03
@sonarqubecloud

Copy link
Copy Markdown

Comment on lines +90 to +101
public showTooltip(event: Event, title: string, description: string): void {
const target = event.currentTarget as HTMLElement;
const rect = target.getBoundingClientRect();
const gap = 8;

this.activeTooltip.set({
top: `${rect.top - gap}px`,
left: `${rect.left + rect.width / 2}px`,
title,
description,
});
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can't this be solved with CSS?
Hide a div and show when hovered on previous item which is the element you want to tooltip to appear on.
I have an example I can give you in the FF reference if you like

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.

Add cvss calculate button

2 participants