Closed MitchellShiell closed 10 months ago
@patrickdsantos Patrick, here is the deploy preview link. Thank you for reviewing it! :D https://deploy-preview-327--overture.netlify.app/
Wonderful work Daniel!! The site is looking AMAZING! Just a few more pieces of feedback below:
Homepage
Make the selected underline static during hover state. Currently the selected line also increases in size during hover. Private Zenhub Image
Make sure all titles of the carousel appear below the image on the left. Private Zenhub Image
Add more padding between the title and grey section. Match the padding for "Getting Started" section. Private Zenhub Image
Increase padding between text and section below. Match the bottom padding for "Getting Started" section. Private Zenhub Image
Hamburger Menu
Website should not peak through when Documentation is expanded. Private Zenhub Image
Is it possible to apply a three column grid to the DMS Bundle Section? Private Zenhub Image
Products Page
Decrease max width of this image. Should not exceed 1000px even in largest screen width. Private Zenhub Image
Same feedback as 7, width of text should not exceed 1000px Private Zenhub Image
Decrease size of icons in mobile view to 50px by 50px and reduce padding between title and text by 10px Private Zenhub Image
Case Studies Page
Top of title should be aligned with the top of the image. Private Zenhub Image
Reduce space between all logo boxes by 15px Private Zenhub Image
Thanks Daniel!
@MitchellShiell The Cell phone view auto zoom and the misalignments, such as the original comments from "Landing Page" section are resolved. This is the issue from the Landing Page section: Private Zenhub Image
@daniel-cy-lu INCREDIBLE WORK!!! Website is nearly perfect!! Thank you for all your hard work - I just have one last piece of feedback.
The following is a mega-ticket for all post-MVP feedback; this can be later distributed into smaller ticket items.
Table of Contents
General
Landing Page
[ ] Center logo, swap hamburger icon to the one used in the mocks, move to left side of the screen and add "Get Started" button to the top right hand corner.
[x] Similar feedback from ticket #289 , accommodate screens that are between 360px and 426px. Right now, when the window is smaller than 426px wide, the hero copy and body copy are no longer centred and starts to cut off.
[x] This was not reflected in the mocks but this area looks a bit squished. Can we increase the padding in between these columns? (try adding additional 10px in between each)
[x] These elements should stack in a single column, refer to mocks for direction.
[x] The spacing on the right side is strange, also reduce size of the image so that it's width is 450px.
[x] Increase margin between arrow and image. Try 30px
[x] Increase padding in between these elements. Try 30px
[x] Increase spacing between each column. Try 30px
[x] Add more padding between illustration and header copy. Try 30px
[x] Reduce padding above to match the bottom of the grey box Private Zenhub Image
Products Page
[x] Double the padding between icon and title (apply to all sections) Private Zenhub Image
[x] Reduce padding between sub-heading and body copy by half (apply to all sections) Private Zenhub Image
[x] There is a subtle grey gradient #F9F9F9 that starts at the top of the blue box and turns into white #FFFFFF (this is applied so there is a visual separation betwen the top of the DMS screenshot and the white background. Refer to mocks for direction. Private Zenhub Image
[x] When the width of the page is >1215px the Arranger icon is smaller than the rest of the other icons. Increase size to match other icons Private Zenhub Image
[x] In mobile view, the Maestro icon does not appear centred. Move icon slightly to the left. Private Zenhub Image
[x] Can't remember if this feedback was already captured but the footer is distorted when the window is >414px Private Zenhub Image
Case Studies Page
Acknowledgements Page
[x] Decrease padding of hero text when screen width is >767px (see image attached). Currently there's 80px of padding plus and additional 50px. Can we reduce so the padding is 50px total?
[x] Accommodate screens that are between 360px and 426px. Right now, when the window is smaller than 426px wide, the hero copy and body copy are no longer centred and starts to cut off. Could we continue to wrap text into a new line until 360px?