Repository navigation
fix(security): keep the estimated severity inside the open leads column - #2465
Conversation
The overview's open leads list gives the badge a fixed w-16 column, and "High · unverified" overflowed it under the title. The column now shows only the estimated severity (dashed outline), and the row ends with the same Unverified chip the findings list uses.
Maple review🟢 Confidence 10/10 · safe to merge. Pure markup change inside one row; the predicate guard and both call sites line up, and recent UI-only remaps of this file shipped without test edits. The Open leads row now renders only the estimated severity in its fixed 64px column and moves the "Unverified" mark to a trailing chip, matching the findings list. Contained view change, safe to merge.
|
|
Caution Review failedThe pull request is closed. Walkthrough
Merge Risk:
|
The Security overview's Open leads list gives the severity badge a fixed
w-16column. An unverified lead renderedHigh · unverified, which overflowed the column and ran under the title.Now the column shows only the estimated severity (the dashed outline already marks it as an estimate), and the row ends with the same
Unverifiedchip the findings list uses. OtherfindingBadgecall sites sit in wrapping rows and are unchanged.Verification
tsrs --noEmit(apps/web): no errors in the touched files. The one error on main,status-page/markdown.tsescapeHtml, is fixed by fix(chat-platform): keep the page block's comment vendor-neutral #2464.oxlint -c .oxlintrc.effect.jsonandoxfmt --checkon both files: clean./lab/security(dark), screenshotted with Playwright. For the shots only, one fixture lead was given acriticalestimate so an unverified lead reaches the top 5. Critical is the longest label, so it is the worst case.Wider layout:
🤖 Generated with Claude Code
Open in [code]smith · Need help on this PR? Tag
@codesmith-botwith what you need.Summary by CodeRabbit