Skip to content

feat: update styles and improve accessibility across frontend#2318

Open
siiddhantt wants to merge 5 commits intoarc53:mainfrom
siiddhantt:feat/standardize-css
Open

feat: update styles and improve accessibility across frontend#2318
siiddhantt wants to merge 5 commits intoarc53:mainfrom
siiddhantt:feat/standardize-css

Conversation

@siiddhantt
Copy link
Copy Markdown
Member

  • Updated text colors to use foreground and muted-foreground for better contrast.
  • Replaced hardcoded colors with theme-based classes for consistency.
  • Enhanced input fields with icons for improved usability.
  • Adjusted button styles for a more cohesive design.
  • Refactored search input components to use consistent styling and layout.
  • Improved layout and spacing in various components for better user experience.
  • Updated tool and source titles and subtitles for clarity.
- Updated text colors to use foreground and muted-foreground for better contrast. - Replaced hardcoded colors with theme-based classes for consistency. - Enhanced input fields with icons for improved usability. - Adjusted button styles for a more cohesive design. - Refactored search input components to use consistent styling and layout. - Improved layout and spacing in various components for better user experience. - Updated tool and source titles and subtitles for clarity.
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 20, 2026

@siiddhantt is attempting to deploy a commit to the Arc53 Team on Vercel.

A member of the Team first needs to authorize it.

@siiddhantt siiddhantt changed the title refactor: update styles and improve accessibility across frontend feat: update styles and improve accessibility across frontend Mar 20, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 47.53%. Comparing base (cc3b174) to head (174dee0).
⚠️ Report is 27 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@ ## main #2318 +/- ## ========================================== + Coverage 47.14% 47.53% +0.38%  ========================================== Files 182 186 +4 Lines 15263 15791 +528 ========================================== + Hits 7196 7506 +310  - Misses 8067 8285 +218 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
Copy link
Copy Markdown
Collaborator

@ManishMadan2882 ManishMadan2882 left a comment

Choose a reason for hiding this comment

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

There are lint failures npm run lint

✖ 779 problems (517 errors, 262 warnings) 517 errors and 0 warnings potentially fixable with the `--fix` option. 
<div key={index} className="relative">
<div
className="bg-gray-1000 dark:bg-gun-metal h-28 cursor-pointer rounded-4xl p-4 hover:bg-[#F1F1F1] dark:hover:bg-[#2C2E3C]"
className="bg-muted dark:bg-answer-bubble h-28 cursor-pointer rounded-4xl p-4 hover:bg-muted dark:hover:bg-muted"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

it replaced specific distinct hover colors with hover:bg-muted on elements whose base is already bg-muted. if intentional please remove the redundant className

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 participants