Skip to content

Conversation

@Wuyika
Copy link
Collaborator

@Wuyika Wuyika commented Feb 10, 2026

Description

  • Dark theme for the statistics screen and all bottom sheets
  • Fix dark theme for the income/expense tab bar on the category screen and transaction form

Type of Change

  • ✨ New feature (non-breaking change which adds functionality)
  • 🛠️ Bug fix (non-breaking change which fixes an issue)
  • ❌ Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 Code refactor
  • ✅ Build configuration change
  • 📝 Documentation
  • 🗑️ Chore

@sourceant
Copy link

sourceant bot commented Feb 10, 2026

Code Review Summary

This PR focuses on enhancing the dark theme support across the statistics screen, bottom sheets, and transaction forms. It replaces hardcoded colors with theme-aware color scheme references.

🚀 Key Improvements

  • Standardized bottom sheet backgrounds to follow the system theme.
  • Refactored DashboardExpenses to include a dynamic legend with correct theme colors.
  • Updated CustomBackButton to support custom colors and better integration in colored headers.

💡 Minor Suggestions

  • Check tab indicator color logic in AddTransactionScreen for consistency.
  • Review withAlpha values to ensure sufficient contrast in dark mode.

Copy link

@sourceant sourceant bot left a comment

Choose a reason for hiding this comment

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

Review complete. See the overview comment for a summary.

Copy link

@sourceant sourceant bot left a comment

Choose a reason for hiding this comment

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

Review complete. See the overview comment for a summary.

Copy link

@sourceant sourceant bot left a comment

Choose a reason for hiding this comment

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

Review complete. See the overview comment for a summary.

Copy link

@sourceant sourceant bot left a comment

Choose a reason for hiding this comment

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

Review complete. See the overview comment for a summary.

Copy link

@sourceant sourceant bot left a comment

Choose a reason for hiding this comment

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

Review complete. See the overview comment for a summary.

Copy link

@sourceant sourceant bot left a comment

Choose a reason for hiding this comment

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

Review complete. See the overview comment for a summary.

Copy link

@sourceant sourceant bot left a comment

Choose a reason for hiding this comment

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

Review complete. See the overview comment for a summary.

Copy link
Contributor

@nfebe nfebe left a comment

Choose a reason for hiding this comment

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

Remove commented code check other sourceant recommendations.

Copy link

@sourceant sourceant bot left a comment

Choose a reason for hiding this comment

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

Review complete. See the overview comment for a summary.

@nfebe nfebe merged commit 7e17399 into dev Feb 10, 2026
4 checks passed
@nfebe nfebe deleted the feat/dark-theme branch February 10, 2026 14:25
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