Skip to content

Fix mods page button colors and table container styling#116

Open
AgentKush wants to merge 2 commits intoDonovanMods:mainfrom
AgentKush:fix/mods-page-ui-bugs
Open

Fix mods page button colors and table container styling#116
AgentKush wants to merge 2 commits intoDonovanMods:mainfrom
AgentKush:fix/mods-page-ui-bugs

Conversation

@AgentKush
Copy link
Copy Markdown
Contributor

@AgentKush AgentKush commented Apr 6, 2026

Summary

  • Download buttons keep their gold bg-icarus-500 but switch from text-white to text-icarus-900 (dark brown) for readable contrast
  • Table container and rows given explicit bg-white dark:bg-slate-900 backgrounds so the layout gradient no longer bleeds through unevenly
  • Content padding tightened (px-3 py-4 md:px-6 md:py-8) so the backing hugs the table more cleanly
  • Added dark:focus:ring-offset-slate-900 to download buttons for correct focus ring in dark mode

Test plan

  • Verify download button text is readable (dark text on gold) in light & dark mode
  • Verify table rows have a clean white/dark background with no gradient bleeding through
  • Verify the outer container backing sits tighter around the table content
  • Click a download button and confirm the focus ring looks correct in both themes
  • Check responsive layout on mobile still looks good

🤖 Generated with Claude Code

Download buttons now use indigo-600 (matching the detail page) instead
of icarus-500, which had poor contrast with white text. Table container
and rows get explicit backgrounds so the layout gradient no longer
bleeds through, and content padding is tightened so the backing hugs
the table more cleanly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Switch button text from text-white to text-icarus-900 so the label
is legible against the bg-icarus-500 gold background. Hover lightens
to icarus-400 instead of darkening.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant