payutc / mozart

Interface web pour l'enregistrement des ventes depuis un écran tactile
4 stars 9 forks source link

Cart management - issues #35, #18 and #43 #45

Closed kofronpi closed 10 years ago

kofronpi commented 10 years ago

Items sent by the server are stored in local storage. A cart, also stored in local storage manages selected items for a given transaction.

User can delete all lines and cancel last operation (remove 1 quantity of the last item clicked).

This pull request solves issues #35, #18 and #43

Next step is transaction management with the server (issue # 37)

apuyou commented 10 years ago

The label on the cancel last operation button is a bit misleading. It does what you are saying, but this is not always the last operation: if I click products A, then B and A again, they will be cancelled in the order A-A-B. Maybe we should plan for a - or delete button on each line?

Anyway, the code is really clean. There are a few things that should be cleaned up later (like some remaining console calls), but this is a development branch so let's go for it.

kofronpi commented 10 years ago

Thank you Arthur, I'll take your comments in consideration for my next pull request. About performances, I'm not really sure since we won't be handling too many articles but I'll look into it. About cancelling, I'll check which solution offers the most user-friendly interface!