Skip to content

Assessments UI#7268

Draft
galvana wants to merge 17 commits intomainfrom
experiment
Draft

Assessments UI#7268
galvana wants to merge 17 commits intomainfrom
experiment

Conversation

@galvana
Copy link
Contributor

@galvana galvana commented Jan 29, 2026

Ticket []

Description Of Changes

Code Changes

Steps to Confirm

Pre-Merge Checklist

  • Issue requirements met
  • All CI pipelines succeeded
  • CHANGELOG.md updated
    • Add a db-migration This indicates that a change includes a database migration label to the entry if your change includes a DB migration
    • Add a high-risk This issue suggests changes that have a high-probability of breaking existing code label to the entry if your change includes a high-risk change (i.e. potential for performance impact or unexpected regression) that should be flagged
    • Updates unreleased work already in Changelog, no new entry necessary
  • UX feedback:
    • All UX related changes have been reviewed by a designer
    • No UX review needed
  • Followup issues:
    • Followup issues created
    • No followup issues
  • Database migrations:
    • Ensure that your downrev is up to date with the latest revision on main
    • Ensure that your downgrade() migration is correct and works
      • If a downgrade migration is not possible for this change, please call this out in the PR description!
    • No migrations
  • Documentation:
    • Documentation complete, PR opened in fidesdocs
    • Documentation issue created in fidesdocs
    • If there are any new client scopes created as part of the pull request, remember to update public-facing documentation that references our scope registry
    • No documentation updates required

Summary of Changes
Assessment view:
Added LLM-generated summaries (lawyer perspective) shown in collapsed headers
Summaries appear on the right next to status tags; risk/confidence stacked below
Improved collapsed section layout and alignment
Made summaries more concise
Removed duplicate risk indicators from left side
Region scoping:
Added region selector in breadcrumb (modal for configuration)
Shows selected regions as tags with flags
Responsive design:
Added breakpoints (992px, 768px, 576px, 480px)
At ≤992px: status container switches to static, stacks vertically
At ≤576px: summary hidden, only risk/confidence and status shown
Fixed overlapping elements on smaller screens
Other:
Increased expanded header height (100px) to prevent squeezing
Removed description from report modal
Fixed padding and spacing throughout
…ration

Visual & UX Improvements
Modern evidence tray redesign: Warmer, Notion-inspired styling with softer borders, improved spacing, and better typography hierarchy
Inline source links: Pill badge-style citation links [1] in questionnaire text that open evidence tray with focused sections
Improved button layout: Right-aligned action buttons in assessment headers with better visual grouping
Slack integration: Added Slack icon button with tooltip in assessments grid for team collaboration
Evidence Tray Enhancements
Removed Card container for initial "Describe the processing" question for cleaner presentation
Removed footer links from evidence items (kept only header links)
Reverted export button to default Ant Design styling
Improved card hover states and transitions
Technical Fixes
Fixed ES module import issues with @ant-design/x by using direct component imports
Added antd and @ant-design/x to Next.js transpile modules
Configured webpack to handle ESM externals and resolve Ant Design version import issues
Fixed Tooltip import to use fidesui instead of direct antd import
Code Quality
All changes maintain existing functionality while improving visual consistency
Responsive design considerations maintained
No breaking changes to existing features
- Remove project name field from Section 2 (redundant with assessment name)
- Redesign controller details: compact inline layout with Edit button (removed click-to-edit hover interactions)
- Simplify questions modal: remove List component and hover effects, display as plain text
- Align modal questions with actual form fields
1. Ran through the FE dev rules so that should pass now

2. Removed unnecessary or dead code

3. Improved the formatting and linting
# Conflicts:
#	clients/package-lock.json
#	clients/package.json
Placing this work behind out feature flag
Privacy Assessments Feature Enhancements:
Assessment Grid (index.tsx):
Added "Create Assessment" modal with name, framework, DPO selection, and auto-population description
Implemented completed assessment state with green left border accent
Completed cards show "Updated {date}" instead of progress bar with green checkmark icon
Added "Completed" filter option to status dropdown
Removed "View all" buttons from template sections
Search bar preserved (even though not yet functional)
Assessment Detail Page ([id].tsx):
Added delete assessment functionality with trash icon and confirmation modal
Consolidated Processing Overview dashboard to single instance at top (removed duplicates from each section)
Dynamic dashboard states: empty states for new assessments, filled data for existing ones
Added confidence indicators (HIGH/MEDIUM) for AI-contributed sections
Contributor tags cleaned up with consistent styling
Reduced bottom padding on grey containers for tighter spacing
Lighter tag colors for Scope section (CORINTH)
Improved section title responsiveness
Completed assessments show all sections as finished
Privacy assessments: evidence drawer, nav order and labels, section header layout

- Open evidence drawer from View evidence; keep questions modal for info icon
- Move Privacy assessments below Privacy requests in left nav
- Nav: sentence case "Privacy assessments" / "Assessments"
- Section headers: View evidence, Request input, Request sent; spacing above buttons
@vercel
Copy link
Contributor

vercel bot commented Jan 29, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
fides-plus-nightly Error Error Feb 5, 2026 9:27pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
fides-privacy-center Ignored Ignored Feb 5, 2026 9:27pm

Request Review

Removed the ability to save drafts.
Removed the location config UX
Moved the "request input from team" button to the top of the page
Added more visual prominence to the "view evidence" button so it doesn't get overlooked.
Clicking on an inline source opens the tray and highlights the source.
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.

2 participants