added downtime minutes and availability view#114
Conversation
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
frontend/src/routes/$incidentId/queries/updateIncidentFieldMutationOptions.ts
Show resolved
Hide resolved
| } | ||
| .bg-avail-bad { | ||
| background-color: #6b1a12; | ||
| } |
There was a problem hiding this comment.
Availability cell text unreadable in light mode
High Severity
The bg-avail-good (#1e6b28), bg-avail-warn (#6b4d0a), and bg-avail-bad (#6b1a12) classes use hardcoded dark background colors that don't vary by theme. In light mode, text-content-headings resolves to near-black (rgb(24, 18, 37)) and text-content-secondary to medium-dark gray, producing near-zero contrast against these dark backgrounds. Availability percentages, downtime text, and incident links become effectively invisible for light mode users.
Additional Locations (2)
Replaces Khanh's table-based availability UI with card-based heatmap bars per region. Each heatmap block links to the incident list with filters pre-applied (region, time range, service tier, status). Also adds Roboto Mono font and moderate-intensity color utilities for the heatmap.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
| {label} | ||
| </Link> | ||
| ); | ||
| } |
There was a problem hiding this comment.
PeriodTab component is unused dead code
Low Severity
PeriodTab.tsx exports PeriodTab but it's never imported anywhere in the codebase. It also duplicates the rendering logic already implemented inline within PeriodTabs.tsx, which is the component actually used by the availability page. This was flagged in the PR discussion as well ("why do we need this defined twice?").
| </div> | ||
| </div> | ||
| ); | ||
| } |
There was a problem hiding this comment.
RegionTable component is unused dead code
Low Severity
RegionTable.tsx exports RegionTable but it's never imported anywhere in the codebase. This is a 157-line component with its own state management, sorting logic, and a duplicated availabilityBgClass threshold function (mirroring getAvailabilityLevel in utils.ts). The availability page only uses RegionCard, not RegionTable.
2 participants
No description provided.