Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🎨 style: Prompt UI Refresh & A11Y Improvements #5614

Merged
merged 26 commits into from
Feb 5, 2025

Conversation

berry-13
Copy link
Collaborator

@berry-13 berry-13 commented Feb 2, 2025

Summary

Closes #5566
Closes #5535

Change Type

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • a11y
  • style update

Testing

Please describe your test process and include instructions so that we can reproduce your test. If there are any important variables for your testing configuration, list them here.

Test Configuration:

Checklist

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • I have commented in any complex areas of my code
  • My changes do not introduce new warnings
  • Local unit tests pass with my changes
  • Any changes dependent on mine have been merged and published in downstream modules.

Changed the useUpdatePromptGroup mutation in prompts.ts to replace the JSON.parse(JSON.stringify(...)) clones with structuredClone. This avoids errors when data contains non‑JSON values and improves data cloning reliability
…djust DashBreadcrumb margin for better alignment
… localization strings for confirmation prompts
…layout and styling; replace button with custom Button component
…s, and Label components for improved user experience
… DashGroupItem components; enhance Dropdown functionality with custom renderValue
…button sizes and localization strings for better accessibility and user experience
…onent; update CategorySelector for improved functionality and accessibility
…iption; enhance layout and accessibility in PromptVariables and PromptForm
…d inputs; improve localization for filter prompts
…mponents; improve layout and styling in PromptForm and CategorySelector
…d components; improve dialog roles and descriptions in SharePrompt and PromptForm
…hance layout and styling in ChatGroupItem, ListCard, and ManagePrompts components
…ty; replace button elements with custom Button component and enhance layout in VariableForm, PromptDetails, and PromptVariables
@berry-13 berry-13 marked this pull request as ready for review February 3, 2025 22:54
berry-13 and others added 3 commits February 4, 2025 23:08
…arePrompt component; enhance readability and maintainability
…mprove conditional rendering for search states and accessibility
@danny-avila danny-avila changed the title feat: prompt UI update 🎨 style: Prompt UI Refresh & A11Y Improvements Feb 5, 2025
@danny-avila danny-avila merged commit 73fe083 into main Feb 5, 2025
4 checks passed
@danny-avila danny-avila deleted the fix/prompt-ui-issues branch February 5, 2025 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants