Conversation
Implement comprehensive UI enhancements based on Atlassian Design System principles: - Add design token system with semantic status colors (success, info, warning, error) and spacing scale for consistent design across light/dark modes - Create Lozenge component for subtle status indicators and tags - Create unified Banner component supporting multiple intents (info, warning, success, error) - Refactor ErrorBanner to use new Banner component - Replace emoji icons with proper Fluent UI icons in ProcessingStatus component - Add Lozenge badges for counts and scores in processing status - Improve visual hierarchy and readability in status components - Enhance QueryInput with smooth transitions and better typography - Add utility classes for interactive hover states and focus rings These improvements enhance visual consistency, accessibility, and user experience while maintaining compatibility with the existing Fluent UI design system.
Replace `any` types with proper TypeScript types in test setup and utility files to improve type safety. Fix ESLint configuration to ignore next-env.d.ts. Apply consistent code formatting across component and test files. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
marevol
force-pushed
the
claude/review-atlassian-design-system-011CV49oUKxCQRCfQdoNBuG5
branch
from
cdf9855 to
6a6d26e
Compare
… configs - Add truncateSnippet function to sanitize and truncate long snippets (UI) - Add NEXT_PUBLIC_SNIPPET_MAX_LENGTH env variable for configurable snippet length - Update Pydantic models from v1 Config class to v2 model_config pattern (API) - Fix test to use content= instead of deprecated data= parameter (API) - Remove redundant @types/dompurify (dompurify has built-in types) - Update tsconfig.json to use jsx: react-jsx and include dev types - Move Fess port mapping from dev to production compose - Add Fess Java opts for crawler cache and highlighting configuration 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…9oUKxCQRCfQdoNBuG5
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Add escapeHtml function to encode meta-characters and prevent XSS when using dangerouslySetInnerHTML with DOM-extracted text. Improve server-side tag stripping with iterative removal to handle malformed or overlapping tags. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implement comprehensive UI enhancements based on Atlassian Design System principles:
These improvements enhance visual consistency, accessibility, and user experience while maintaining compatibility with the existing Fluent UI design system.
Description
Type of Change
Related Issue
Fixes #(issue)
Changes Made
Testing
API Tests
pytest)UI Tests
npm test)npm run test:e2e)Manual Testing
Checklist
Screenshots (if applicable)
Additional Notes
Deployment Notes
For Reviewers: