-
Notifications
You must be signed in to change notification settings - Fork 1
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
Review Mobile Figma Wireframe and then Proceed to Desktop Site Wireframe #29
Comments
@ctrocc obviously plenty of polish to do but overall looks pretty good as a layout! Should I avoid "polish" type feedback like better spacing/margin/coloring and focus on information hierarchy? Or do you plan on polishing it to match the branding/style patterns as well? @stefanny1mc can probably help with the polishing kind of work at some point soon (currently working on another project for a few weeks at least) but for now I will focus on layout feedback |
Yes I can help with this, no problem @otech47 |
left a couple comments and some copy samples @ctrocc |
Sounds good, I think for now its best to focus on IA and then once it's mostly put together, the three of us can go through and polish together. |
https://www.figma.com/file/KlLr27PTO7Dzz3D9CE2LzT/setlife?node-id=0%3A1
The text was updated successfully, but these errors were encountered: