Skip to content

Redesign flashcard deck workflows#53

Merged
AliBenrami merged 6 commits into
developfrom
Ali/issue-41-flashcards-ux
May 30, 2026
Merged

Redesign flashcard deck workflows#53
AliBenrami merged 6 commits into
developfrom
Ali/issue-41-flashcards-ux

Conversation

@AliBenrami

@AliBenrami AliBenrami commented May 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Redesigns /flashcards into focused library, create, review, and edit flows.
  • Adds preview-before-save generation, manual card editing, deck update, and deck deletion contracts.
  • Centralizes flashcard deck normalization and editable deck validation.
  • Fixes the rebased markdown code renderer path so existing note-renderer specs stay green with the current highlighter return shape.

Closes #41

Validation

  • pnpm test
  • pnpm lint
  • pnpm exec tsc --noEmit
  • pnpm build
  • GitHub PR checks: lint, Vercel, Vercel Preview Comments all passed.
  • Browser smoke: /flashcards redirects through auth and renders /login after local dependency install.

@vercel

vercel Bot commented May 27, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
task-master Ready Ready Preview, Comment May 30, 2026 1:52am

@AliBenrami AliBenrami marked this pull request as ready for review May 30, 2026 00:01
@AliBenrami AliBenrami requested a review from hamizfaraz as a code owner May 30, 2026 00:01
…t polish

- Make entire library card clickable for primary Take/Review action
- Move Edit/Delete (and Duplicate for quizzes) behind a three-dot MoreHorizontal menu with backdrop dismiss
- Fix quiz edit section scrolling in one-pager layout
- Relax sourceNoteTitles Zod validation (.min(1) → .default([])); catch parse errors server-side and return generic message to client
- Replace inline error banners with toast.error() calls in flashcards client
- Remove explanatory page headers from both quizzes and flashcards
- Align library view structure and background between empty/filled states
- Simplify quiz cards to name, date, badges only (remove question previews, attempt history)
- Narrow card grids to md:grid-cols-2 xl:grid-cols-3 with auto-rows-min/content-start
- Expand AGENTS.md with UI copy, component reuse, React best practices, and error UX guidelines

Co-authored-by: Cursor <[email protected]>
@AliBenrami AliBenrami merged commit 18e94b1 into develop May 30, 2026
3 checks passed
@AliBenrami AliBenrami deleted the Ali/issue-41-flashcards-ux branch May 30, 2026 01:53
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.

Feature Request: Redesign Flashcard Experience and Creation Flow

1 participant