Skip to content

UI fixes for asset avatar for light theme - #414

Merged
Kukks merged 3 commits into
masterfrom
fix-asset-avatar
Mar 13, 2026
Merged

Kukks merged 3 commits into
masterfrom
fix-asset-avatar

Conversation

@pietro909

@pietro909 pietro909 commented Mar 12, 2026 •

Copy link
Copy Markdown
Contributor

Fixes for light theme:

  • maintain circle shape for avatar
  • asset card background and ticker color
Sad Wilma Happy Wilma
Screenshot from 2026-03-12 16-09-37 Screenshot from 2026-03-12 16-07-04
Screenshot from 2026-03-12 16-14-18 Screenshot from 2026-03-12 16-17-57

Summary by CodeRabbit

  • Style
    • Improved avatar component styling with consistent circular clipping and sizing.
    • Updated token ticker text color for better visibility.
    • Adjusted purple background color in the theme palette.

@pietro909
pietro909 requested a review from bordalix March 12, 2026 15:23
@coderabbitai

coderabbitai Bot commented Mar 12, 2026 •

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 49c61a02-72f7-491a-9f74-5aa705a62654

📥 Commits

Reviewing files that changed from the base of the PR and between fda01a1 and d576330.

📒 Files selected for processing (3)
  • src/components/AssetAvatar.tsx
  • src/components/AssetCard.tsx
  • src/ionic.css

Walkthrough

The changes include styling and layout refinements to the avatar component with a reusable style object, a text color adjustment in the asset card, and a purple background color update in the theme variables.

Changes

Cohort / File(s) Summary
Component Styling Refinements
src/components/AssetAvatar.tsx, src/components/AssetCard.tsx
Avatar component refactored with centralized avatarStyle object; icon images now wrapped with overflow clipping and stretched to fill container. Token ticker text color changed to white.
Theme Configuration
src/ionic.css
Purple background color variable updated from #391998 to #24154f.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • tiero
  • bordalix
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main changes: UI fixes for the asset avatar and related styling for light theme support, which aligns with the three key file modifications in the changeset.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-asset-avatar
📝 Coding Plan for PR comments
  • Generate coding plan

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.

@Kukks
Kukks merged commit 60651a8 into master Mar 13, 2026
5 checks passed
@Kukks
Kukks deleted the fix-asset-avatar branch March 13, 2026 08:38
Kukks added a commit that referenced this pull request Mar 13, 2026
Merge master into wt-navbar-redesign-20260226, incorporating:
- nak version fix (#420)
- password navigation fix (#419)
- chain swaps restore (#415)
- asset avatar UI fixes (#414)
- approved asset icons (#416)
@coderabbitai coderabbitai Bot mentioned this pull request Apr 22, 2026
4 tasks
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.

3 participants