Skip to content

Comments

Get started fix#400

Merged
n13 merged 3 commits intomainfrom
get_started_fix
Feb 21, 2026
Merged

Get started fix#400
n13 merged 3 commits intomainfrom
get_started_fix

Conversation

@n13
Copy link
Collaborator

@n13 n13 commented Feb 21, 2026

As per figma, get started section

Simulator Screenshot - iPhone 16e - 2026-02-21 at 13 56 15

@n13
Copy link
Collaborator Author

n13 commented Feb 21, 2026

The PR Get started fix looks good. It refactors the ActivitySection to be stateful, allowing for an expandable "Get Started" section and a proper empty state for transactions.

Summary of Changes

  1. State Management: Converted ActivitySection to ConsumerStatefulWidget to handle the _getStartedExpanded state.
  2. New "Get Started" UI:
    • Added an expandable section with AnimatedCrossFade for smooth toggling.
    • Links ("Get Testnet Tokens", "Community", "Tech Support") are now styled in a container with dividers and arrow_outward icons.
  3. Empty State: Added a dedicated _emptyState widget ("No Transactions Yet") that appears when there are no transactions, improving the user experience for new accounts.

Code Quality

  • Clean & Readable: The code is well-structured and follows Flutter conventions.
  • Correctness: The logic for toggling the section and displaying the empty state is sound.
  • No Linter Errors: The file is free of linter issues.

The changes align with the PR description ("As per figma") and the "better arrow" commit. I recommend merging this PR.

Copy link
Collaborator

@dewabisma dewabisma left a comment

Choose a reason for hiding this comment

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

LGTM!

@n13 n13 merged commit a31977b into main Feb 21, 2026
1 check passed
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