PA Review: Study - Githubissues

xnanodax commented 6 years ago

[x] Backend: DB, model, controller, views
[x] Redux Loop: ajax, actions, reducer
[x] Presentational Components
[ ] Adequate styling
- [ ] Smooth, bug-free navigation
- [ ] Seeded with info to demonstrate this feature

xnanodax commented 6 years ago

What is the best way to keep track of mastery of a deck and mastery progress of a deck?

LochandQuay commented 6 years ago

@xnanodax

Console Errors:

[x] Invalid DOM property class. Did you mean className? When clicking on Study.

Functionality Issues:

[ ] If I click done, it should send me back to the deck show, since that's where I was coming from when I clicked study.

Styling Issues:

[ ] The "# / #" part goes outside the circle. Maybe you can do something to fix the size of the div to be within the progress circle and shrink the font accordingly? OR just make the font smaller.
[ ] Flashcard should probably shrink a bit (dynamically). It gets covered if I have DevTools open on a laptop.
[ ] Reveal answer button now centered horizontally.

xnanodax commented 6 years ago

After someone studies the deck, they'll hit a complete page and from there they can choose whether to go back or search for other decks!

xnanodax / brain-map