Skip to content

Match study empty state to design + Browse examples → Discover - #17

Merged
jvsena42 merged 3 commits into
mainfrom
fix/study-screen-empty-state
Jun 20, 2026
Merged

Match study empty state to design + Browse examples → Discover#17
jvsena42 merged 3 commits into
mainfrom
fix/study-screen-empty-state

Conversation

@jvsena42

Copy link
Copy Markdown
Owner

Summary

Aligns the Study tab empty state with Pencil design node nwHYV and wires the Browse examples action to the Discover tab.

Design match (nwHYV)

  • Circular 100dp accentPrimarySoft badge with a book-open icon (Material MenuBook, 48dp, accentPrimary) — replacing the rounded-square 📚 emoji badge.
  • Card padding 24h/32v, item gap 16, subtitle line height 21sp (design's 1.5×).
  • Copy updated: "Nothing to study yet", study-oriented subtitle, and "Create a deck" primary button.
  • Empty state now centers the card + actions in the space below the greeting (design's centerWrap) instead of being top-aligned in the scroll column.

Navigation

  • Browse examples now switches the pager to the Discover tab. The effect was already plumbed through HomeRoute/HomeViewModel; only the destination lambda in MainScreen was missing.

Notes

  • Uses the native Material MenuBook icon rather than Lucide book-open, per the project's native-first rule (the app already relies on Material icons).

Verification

  • ./gradlew :composeApp:compileDebugKotlin
  • ./gradlew detektAll

🤖 Generated with Claude Code

jvsena42 and others added 3 commits June 20, 2026 12:08
Align the Study tab empty state with Pencil node nwHYV: circular book-open
badge (Material MenuBook) in place of the rounded-square emoji, design
padding/gap/line-height, updated title/subtitle/primary-button copy, and
vertical centering of the card + actions in the space below the greeting.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Wire the home empty-state 'Browse examples' action to switch the pager to
the Discover tab. The effect was already plumbed through HomeRoute/HomeViewModel;
only the destination lambda in MainScreen was missing.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@jvsena42
jvsena42 enabled auto-merge June 20, 2026 15:19
@jvsena42
jvsena42 merged commit 262dba9 into main Jun 20, 2026
2 checks passed
@jvsena42
jvsena42 deleted the fix/study-screen-empty-state branch June 20, 2026 15:20
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.

1 participant