Skip to content

Fix UI polish review feedback: accessibility, consistency, and UX improvements#10

Open
Copilot wants to merge 4 commits intoclaude/polish-ui-design-S5CS6from
copilot/sub-pr-7
Open

Fix UI polish review feedback: accessibility, consistency, and UX improvements#10
Copilot wants to merge 4 commits intoclaude/polish-ui-design-S5CS6from
copilot/sub-pr-7

Conversation

Copy link
Contributor

Copilot AI commented Jan 27, 2026

Addresses review comments from PR #7 covering accessibility gaps, animation inconsistencies, misleading UX elements, and branding conflicts.

Accessibility

  • Added aria-label to topic chips: aria-label="Search for ${topic.label} videos"
  • Added prefers-reduced-motion support: motion-reduce:animate-none motion-reduce:opacity-100

Button Component Consistency

  • Fixed gradient animation gap in danger/success variants
  • Added missing hover:bg-[position:100%_0] to match primary button behavior

UX Fixes

  • Suggested topics: Opens YouTube search in new tab instead of setting invalid search URL as input value
  • Video preview card: Added "EXAMPLE OUTPUT PREVIEW" label, removed non-functional play button
  • Navigation arrow: Added group class to button for proper group-hover:translate-x-1 behavior

Code Quality

  • Consolidated duplicate shimmer animations (skeleton now uses shared keyframe)
  • Removed unused CSS classes: .input-glow, .chip, .text-shadow-lg, .text-shadow-glow
  • Simplified animation delays (removed unnecessary CSS custom property wrapper)

Branding

  • Reverted "Video2Learn" → "UVAI.io" for consistency with layout.tsx metadata, dashboard, and playground
// Before: Sets invalid URL
onSelectTopic={(query) => {
  setVideoUrl(`https://www.youtube.com/results?search_query=${query}`);
}}

// After: Opens search in new tab
onSelectTopic={(query) => {
  const searchUrl = `https://www.youtube.com/results?search_query=${encodeURIComponent(query)}`;
  window.open(searchUrl, '_blank', 'noopener,noreferrer');
}}

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@vercel
Copy link

vercel bot commented Jan 27, 2026

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

Project Deployment Review Updated (UTC)
event-relay-web Error Error Jan 27, 2026 10:02pm
v0-uvai Error Error Jan 27, 2026 10:02pm

Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
…olidate shimmer

Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
Copilot AI changed the title [WIP] Polish UI design with enhanced visual components Fix UI polish review feedback: accessibility, consistency, and UX improvements Jan 27, 2026
Copilot AI requested a review from groupthinking January 27, 2026 22:04
@groupthinking groupthinking marked this pull request as ready for review January 28, 2026 03:34
Copilot AI review requested due to automatic review settings January 28, 2026 03:34
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request addresses UI polish review feedback from PR #7, focusing on accessibility improvements, button component consistency, UX refinements, code quality, and branding alignment. The changes enhance the user experience by adding proper ARIA labels, supporting reduced motion preferences, fixing misleading interactions, and consolidating duplicate CSS.

Changes:

  • Added accessibility features including aria-labels for topic chips and prefers-reduced-motion support for animations
  • Fixed button component gradient animation inconsistencies in danger/success variants
  • Improved UX by opening YouTube searches in new tabs instead of setting invalid URLs, removing non-functional play button, and fixing navigation arrow hover behavior
  • Cleaned up unused CSS classes and consolidated duplicate shimmer animations
  • Standardized branding from "Video2Learn" to "UVAI.io" throughout the application

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
apps/web/src/components/ui/SuggestedPrompts.tsx Added aria-label and prefers-reduced-motion support to topic chips and animations
apps/web/src/components/ui/Button.tsx Added missing gradient animation hover states to danger and success button variants
apps/web/src/app/page.tsx Fixed suggested topics to open YouTube search in new tab, added preview label, fixed navigation arrow group behavior, updated branding to UVAI.io
apps/web/src/app/globals.css Removed unused CSS classes (input-glow, chip, text-shadow utilities) and added prefers-reduced-motion support for skeleton loading

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