@HanselChavezSubmitted over 1 year ago
David Mclean
@davuplsAll comments
- @davuplsPosted over 1 year ago
Great job with the layout of your design it looks good on all sizes over on my computer.
CSS 👨🎨
- If you intentionally used Internal CSS then that's okay, otherwise, I would recommend external.
- The design looks so good, I would put the last effort into getting the button shadows. Here is a CSS Shadow Generator to make it easier: Box Shadow Website
Marked as helpful1 - @AbelardoOkSubmitted over 1 year ago
I can't make the contact container be upper than footer, and i have some issues in the project, but i try make the best i can to do this landing page in React/NextJs and tailwindCss
@davuplsPosted over 1 year agoI would review the website at tablet size: 810w-1080h. The testimonials are squeezed, especially the "Get early access today" section does not look good at all. Looks like your using React, Tailwind etc, which I don't know to tell you what went wrong. Keep up the good work!
1 - @rh0seSubmitted over 1 year ago
I would appreciate any feedback on this project
@davuplsPosted over 1 year agoI think you did great on this one. Maybe make the alt text more descriptive for example: "sedan icon". An class name as well example: "more-btn" / "more-link". But good work I understood your code at a quick glance.
Marked as helpful0 - @davuplsSubmitted over 4 years ago
Was using CSS Grid on <section class="tags-container"> to control the layout a valid approach?
What topics do you notice right away I could improve on?
Is my Code readable to you?
Recommendations?
