Skip to content

refactor: replace _SessionCard with PracticeChantListTile for improve…#488

Merged
tentamdin merged 1 commit into
developfrom
fix/InconsistentUI
Jul 3, 2026
Merged

refactor: replace _SessionCard with PracticeChantListTile for improve…#488
tentamdin merged 1 commit into
developfrom
fix/InconsistentUI

Conversation

@dhakar66

@dhakar66 dhakar66 commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

…d UI consistency and functionality

@dhakar66 dhakar66 requested a review from tentamdin July 2, 2026 18:08
@greptile-apps

greptile-apps Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Confidence Score: 4/5

Safe to merge; the refactor is mechanical and the shared widget is already in use across other screens with no new logic paths introduced.

Both changed files are straightforward: the widget gains opt-in flags with backward-compatible defaults, and the screen swaps an inline card for the shared widget. The only worth-noting side effect is the description text truncating at 2 lines instead of 3 in the chants tab — whether intentional or accidental, it is worth a quick check before landing.

practice_chant_list_tile.dart — verify the maxLines: 2 cap for description text is the intended behaviour for the chants session-picker context.

Reviews (1): Last reviewed commit: "refactor: replace _SessionCard with Prac..." | Re-trigger Greptile

@tentamdin tentamdin merged commit 34c76c2 into develop Jul 3, 2026
1 of 2 checks 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