-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Launch Checklist #19
Comments
Here is a test sequence:
I had to fix the order flow again after the IE fixes so did not get to stripe integration which will come Wed. Also still need to send order receipt and order arrived emails. There is still a listener leak in admin but it should work if you refresh once in a while. Please go through the sequence and send feedback/changes/problems. |
|
|
|
|
just went through the flow again. quick feedback:
|
|
CONTENT THAT NEEDS TO BE FINALIZEDSUNDAY: we'll go through this to-do list after UI update has been pushed@omflash @jmsantha @gilliansl @jrf0110
Text
Web: Restaurant Name, Menu Item Names, & Pricing
Web: Menu Page
Web: Menu Page: TIMES UP
Web: Checkout
Web: Order Success Page
|
|
@cadorn @jrf0110
I also just found this:
|
|
|
|
"To give feedback or ask questions, please email us at [email protected]" |
This issue has been split into two which I am about to prioritize as a start: |
This list HAS MOVED TO #41 (comment)Prioritized functionality:
After that ^^ next things to do:
|
'Tis a design constraint since all tiles need to be the same height. We could go a tiled layout somewhat like Pinterest, but this layout isn't supported by standard CSS and adds a bit of dev overhead. Definitely doable. Another option is to have variable height, but the height of an entire row would stay consistent. This, however, removes our ability to add a breakpoint for medium width devices, making the item tiles go to two to a row, rather than three |
response to john's comment ^^ I see. What do you guys think about making a hover state for the items which will show the image, longer title, and description? Wouldn't work for mobile of course. My main issue is that user's aren't getting a complete picture of what the item is. BUT, this isn't a HUGE deal right now.. |
$feature #1
lunchroom.goodybag.com
to heroku app @jrf0110TipsThe text was updated successfully, but these errors were encountered: