Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions docs/implementation-notes.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ The current slice is a responsive React/Vite local-first review workspace, a det
- The current local workspace is persisted in browser storage under a versioned key, including sources, findings, analysis metrics, review notes and decisions. This is intentionally device-local and does not replace a shared backend.
- Review reports and approved change requests can be exported as Markdown files without sending project data to an external service.
- Pilot safeguards include a 10 MB source limit, required scope/message source validation, source reclassification and removal, stable finding IDs, basic negation and sender-role handling, included/excluded scope matching, and multipart email/plain-text Slack fallbacks.
- The pilot visual system remains the original warm paper/ink/orange workspace design; the page header and demo banner have explicit layout styles so pilot-state additions do not fall back to browser defaults.

## Next technical slice

Expand Down
7 changes: 7 additions & 0 deletions reports/browser-qa.md
Original file line number Diff line number Diff line change
Expand Up @@ -79,3 +79,10 @@ PASS for the local pilot slice. The application is ready for a controlled export
- The source list showed both files, the status changed to `Ready to analyse`, and the analysis completed with one finding.
- No console errors occurred during the full flow.
- The onboarding dialog receives focus when opened and remains dismissible with `Escape`.

## 2026-08-03 visual regression pass

- Restored the missing desktop page layout rules for `.content-wrap`, `.page-intro`, headings, intro copy and action buttons.
- Added the visual treatment for the `Demo data` status banner.
- Verified the local desktop render at 1280 × 720 against the intended paper/ink/orange design system.
- Console errors and warnings were empty after the fix.
41 changes: 41 additions & 0 deletions reports/design-audit.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
# ScopeGuard design audit

Date: 2026-08-03

## Finding

The pilot hardening change introduced new page markup but dropped the existing CSS block for the editorial page header. The sidebar and lower workspace styles remained active, while the intro layout, heading scale, action buttons and demo banner fell back to browser defaults. This was a high-severity visual regression because the primary workspace lost its visual hierarchy.

## Before / after

- Before: the main content started at the page edge, the headline and buttons were unstyled, and the demo notice read as plain text.
- After: the original centered content rail, large editorial headline, paper/ink/orange controls and bordered demo banner are restored.

## Checked aspects

1. Page background and warm paper palette — pass.
2. Sidebar width and rail separation — pass.
3. Content max width and horizontal padding — restored and pass.
4. Editorial headline scale and tracking — restored and pass.
5. Intro copy width and line height — restored and pass.
6. Primary and quiet action button states — restored and pass.
7. Demo banner hierarchy and contrast — added and pass.
8. Analysis status strip placement — pass.
9. Evidence/source two-column grid — pass.
10. Finding card borders, radius and spacing — pass.
11. Accent orange reserved for risk and actions — pass.
12. Responsive rules remain present for mobile layout — pass by source inspection; mobile browser emulation remains a follow-up check.
13. Focus-visible treatment remains present — pass.
14. Reduced-motion rule remains present — pass.

## Files changed

- `src/styles.css`
- `reports/browser-qa.md`

## Validation

- `npm run typecheck` — passed.
- `npm run build` — passed.
- Local browser render at 1280 × 720 — restored visual hierarchy.
- Browser console errors and warnings — none observed.
2 changes: 2 additions & 0 deletions src/styles.css

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading