-
Notifications
You must be signed in to change notification settings - Fork 4
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
component: individual intent casting #49
Comments
TODO:
NICE TO HAVE:
|
Initial UI for member intent casting for a product: Three sliders with dashed lines:
|
@sarah-arrrgh can you add some text explaining the image? |
Design questions for anyone:
Not expecting a change to what's being done - rather documentation of why we are doing it this way. That way we have documentation to reference when we test via showing the dumb component to peeps via storybook. |
@agentlewis have added some notes to the photo comment, let me know if you are still unclear on anything! |
Great thanks Design questions that remain unclear to me:
New things:
|
TODO:
|
Question:
|
@sarah-arrrgh here's a draft attempt: #146 (comment). what do you think? |
End of day/week report:
|
@sarah-arrrgh to me the view on the left looks like a mobile view and the right the view on desktop. The view on the rleft reminds me of a card and I can imagine a list of cards (mobile) / grid of cards (desktop) to show items with a visually rich expierience. Is this what you were thinking? |
@agentlewis the view on the left is the product list view - yes a grid of cards like you are imagining. and the view on the right is single product view, so has more info on the product including price points and facets. |
@sarah-arrrgh are you approaching this mobile first or skipping mobile? |
End of day report:
TODO:
|
@agentlewis hmmm haven't been doing this mobile first |
This is going to come up very quickly as people will say "does it work on mobile". My personal opinion is that it is easier and more effective to develop from mobile to desktop rather then the other way round. Nvm to much - functionality is more important. I'd suggest you break out a ticket to mobilise what you have done so far. We don't need it for the 27th but will before a beta test. |
@sarah-arrrgh see above |
creating the individual intent casting view so it can be tested and tweaked
#20
The text was updated successfully, but these errors were encountered: