Skip to content

Revert "fix(ui): use distinct color palette for group tags" - #2832

Merged
Calcium-Ion merged 1 commit into
mainfrom
revert-2759-fix-group-colors
Feb 4, 2026
Merged

Revert "fix(ui): use distinct color palette for group tags"#2832
Calcium-Ion merged 1 commit into
mainfrom
revert-2759-fix-group-colors

Conversation

@seefs001

@seefs001 seefs001 commented Feb 4, 2026

Copy link
Copy Markdown
Collaborator

Reverts #2759

Summary by CodeRabbit

  • Bug Fixes

    • Updated tag color derivation to improve consistency across the application.
  • Refactor

    • Simplified internal color palette logic for tag rendering.

@coderabbitai

coderabbitai Bot commented Feb 4, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

Walkthrough

The PR modifies web/src/helpers/render.jsx to simplify color handling in tag rendering. It removes the previously exported groupToColor function and its supporting groupColors constant, replacing their usage with the existing stringToColor function. Minor formatting adjustments are applied to filter functions. Net change: -43 lines, +15 lines.

Changes

Cohort / File(s) Summary
Color Helper Refactoring
web/src/helpers/render.jsx
Removed groupToColor(str) function and groupColors constant. Updated renderGroup to use stringToColor(group) for color derivation. Applied minor formatting adjustments (spacing, line breaks) to model-category filter functions.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 Out with the old, away with the hues,
One function sufficed—no need for the blues!
Colors now unified, simpler the way,
Less code to maintain means hooray, hooray! 🎨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch revert-2759-fix-group-colors

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Calcium-Ion
Calcium-Ion merged commit 564f407 into main Feb 4, 2026
1 check was pending
@Calcium-Ion
Calcium-Ion deleted the revert-2759-fix-group-colors branch March 17, 2026 09:25
ennnnny pushed a commit to ennnnny/new-api that referenced this pull request Mar 17, 2026
…group-colors

Revert "fix(ui): use distinct color palette for group tags"
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