Skip to content

Commit

Permalink
Merge pull request #611 from cisagov/610-scrolling-functionality-on-m…
Browse files Browse the repository at this point in the history
…ost-common-vulnerabilities-table-crasm-605

Resolved lack of scroll in VulnerabilityCard
  • Loading branch information
schmelz21 authored Sep 18, 2024
2 parents 7d07e53 + fb2790c commit adc83bd
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
5 changes: 2 additions & 3 deletions frontend/src/pages/Risk/VulnerabilityCard.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -56,9 +56,8 @@ const VulnerabilityCard = (props: {
title: string
) => history.push(filteredVulnTableLink, { title: title });

const vulnerabilityGridContent = data.slice(0, 4).map((vuln) => {
const filteredVulnTableLink =
'/inventory/vulnerabilities?sort=createdAt&desc=false';
const vulnerabilityGridContent = data.slice(0, 10).map((vuln) => {
const filteredVulnTableLink = '/inventory/vulnerabilities';
const ariaLabel = `Details link to the vulnerabilities table filtered to ${vuln.title}.`;
const onRowKeyDown = (event: React.KeyboardEvent) => {
if (event.key === 'Enter' || event.key === ' ') {
Expand Down
2 changes: 1 addition & 1 deletion frontend/src/pages/Risk/style.ts
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ export const RiskRoot = styled('div')(({ theme }) => ({
'& h3': {
textAlign: 'center'
},
overflow: 'hidden'
overflow: 'auto'
},
[`& .${classesRisk.chartSmall}`]: {
height: '85%'
Expand Down

0 comments on commit adc83bd

Please sign in to comment.