Skip to content

fix(ui): scan page cleanup and restore layout spacing#3643

Merged
gorkem-bwl merged 1 commit intodevelopfrom
fix/ui-scan-page-and-layout-spacing
Apr 1, 2026
Merged

fix(ui): scan page cleanup and restore layout spacing#3643
gorkem-bwl merged 1 commit intodevelopfrom
fix/ui-scan-page-and-layout-spacing

Conversation

@gorkem-bwl
Copy link
Copy Markdown
Contributor

Summary

  • Remove card border/background around scan form on AI Detection scan page
  • Add 16px padding above Repository URL block
  • Narrow the repository URL input field (max-width 480px)
  • Show stat cards regardless of scan count
  • Restore 16px gap between children in PageHeaderExtended (was lost when outer Stack was set to gap=0 for alert layout fix)
  • Add conditional bottom margin after summary cards

Test plan

  • Check /ai-detection/scan — no card around form, narrower input, stats visible with 0 scans
  • Check /ai-incident-managements — spacing between cards, filter row, and table is 16px
  • Check /vendors and /policies — no extra spacing between description and tab bar
  • Check /risk-management — summary cards have 16px gap before filter row

## Changes
- Remove card border/background around scan form on AI Detection scan page
- Add 16px padding above Repository URL block
- Narrow the repository URL input (max-width 480px)
- Show stat cards regardless of scan count (always visible)
- Restore 16px gap between children in PageHeaderExtended
- Add 16px bottom margin after summary cards (only when cards exist)

## Context
The outer Stack in PageHeaderExtended had gap=0 to prevent alerts from
affecting layout, but this also removed spacing between page children.
Wrapped children in a Stack with gap=16px and added conditional bottom
margin on the summary cards section.
@gorkem-bwl gorkem-bwl merged commit d887d86 into develop Apr 1, 2026
3 of 4 checks passed
@gorkem-bwl gorkem-bwl deleted the fix/ui-scan-page-and-layout-spacing branch April 1, 2026 04:20
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.

1 participant