search

s360 / StudentSuccessLink-Website

MIT License
0 stars 1 forks source link

Design Review #3

Open levana360 opened 6 years ago

levana360 commented 6 years ago

For sslmarketing.dev.s360.is , here is the full link for the design review https://preview.uxpin.com/c0b7763a693c150ed360c20878876f6cbd9cbadc#/pages/79757630/comment/no-panels

  1. Add "Benefits" menu in the navigation bar, after the "Features" menu

image

  1. Change "About the project" to "About" image

  2. adjust padding and placement according to the design current version: image

design version: (https://preview.uxpin.com/c0b7763a693c150ed360c20878876f6cbd9cbadc#/pages/79146494/comment/no-panels) image

  1. Change the line opacity current: image

design: image

  1. Adjust placement so this element centers

current: not in the center image

design: image

image

example of current one on the top:

image

  1. In the quotes section, reduce font size down one px or em

image

  1. Darken this background a bit so there's greater contrast for the white text image

  2. Hide the logos for now. We will post them again in a few weeks image

  3. Create a white border around the pictures, maybe 5 px? Please explore an option that looks good. The thumbnails can scale down smaller with a border. image

mkom commented 6 years ago

update, please double check

hafizhahhanifa commented 6 years ago

1 & 2. Done, update menu https://screencast-o-matic.com/watch/cFVTiCoT9F

  1. Done, adjust padding 2018-01-25_1449

  2. Done, line 2018-01-25_1501

5 & 6. Done, right now the quotes section not jumping. Also, the image has been added. It would be better if the images are square shaped so do not change its size https://screencast-o-matic.com/watch/cFVTi9oT9k Note: please refresh the site when you resize the browser

7 & 8. Done, darken background and hide the logos 2018-01-25_1423

  1. Hi @levana360 rather than a white border, our developer suggests using cyan color. So the border is more visible. Is this okay or we still use the white border?

Here's the preview of both color:

levana360 commented 6 years ago

  1. @hafizhahhanifa for the border, the Cyan color looks good! Make sure the width is the same though: image

  2. For the Quotes section, please change the images to the ones attached in mavenlink. Match the faces according to their names as well image

mkom commented 6 years ago

fix and update the border

hafizhahhanifa commented 6 years ago

  1. Done, fix border https://screencast-o-matic.com/watch/cFVOijot6k

  2. Done, update image https://screencast-o-matic.com/watch/cFVOi6ot6C