Hello! This is my first time making a landing page. I experimented a little bit with my scss and tried to automate my custom properties. I have more documentation in the github repo. Tell me what you think!
Reinhardt Venter
@RingoFloydAll comments
- @franco-aSubmitted over 3 years ago@RingoFloydPosted over 3 years ago
Great design Franco, I also submitted the same challenge recently - my first submission. I'm not really satisfied with mine, so I am going to be having a look at your code to see where I can improve my own, hope you don't mind :)
1 - @MuhammadHassaanMahmood5550Submitted over 3 years ago
will wait for your feedback and suggestions.π
@RingoFloydPosted over 3 years agoHi Hassaan, looks great. The only suggestion I have is purely from a visual perspective - you can maybe make the text of your headings a bit larger. For instance your h1 can be larger, and your h2 and h3 is very similar. Maybe look at making the h2 stand out a bit more.
0 - @Aditya798-createSubmitted over 3 years ago
Can someone review it
@RingoFloydPosted over 3 years agoHi Aditya, I like your solution, I think you've done very well. I am missing the curve divider between the sections
class="introduce"
andclass="services"
. The design creates a mountain range outline, which breaks the straight lines quite a bit, so even though it is a small detail I do think it is worthwhile spending time on it.I hope you find my comments useful, I am still a newbie, so feel free to ignore if you feel it is not relevant. :)
0