I don't know JavaScript so there won't be any of that there. Any feedback would be appreciated.
Harold
@harika09All comments
- @soodaayushSubmitted almost 4 years ago@harika09Posted almost 4 years ago
Good job on this one.
some recommendation to improve it.
- try not fix the illustration image when display on bigger sizes display it will zoom it. 2.add spacing on your navigation menu. 3.add spacing on the latest articles card.
- make it responsive.
overall this was great! Happy coding!
0 - @GerriEzeochaSubmitted almost 4 years ago
This project was my first exposure to FrontEnd Mentor! I worked through the YouTube Series "Build a website from scratch" by Jessica Chan. She is a great teacher. I had some issues loading my images for my hero section, I had to move them to the same root folder holding my sass/style sheets.
All in all, I learned A LOT working on this project, and hopefully can apply those to future projects.
@harika09Posted almost 4 years agoGreat job on this one @Gerri Ezeocha.
just to remove the min-height: 350vh; to remove the white spaces after the attribution.
Happy Coding!
0 - @StelinUwUSubmitted almost 4 years ago
❤
@harika09Posted almost 4 years agoGreat Job on this one @Stelin
Got some few recommendation.
- check the validation when I clicked the button it will just stack the error.
- add cursor: pointer; when hover on the button.
- lower the padding on the smaller device on class="info".
Good job on this. Happy coding!
1 - @irfan-fauziSubmitted almost 4 years ago
feedback please
@harika09Posted almost 4 years agoNice work with this. I will just add some recommendation.
- try to center the illustration-img to center with it its max-with: 900px
- try to center to footer just like the design.
Happy Coding!
0 - @nelsonuprety1Submitted almost 4 years ago
Doesnt look so good.
@harika09Posted almost 4 years agoNice work with this.
try to add the details, box and like on a container Example <main class="container"> <div class="details">...</div>
<div class="box">...</div> <div class="like">...</div> </main>also add no-repeat on the background image
then on the css add 100vh on the main
0 - @harika09Submitted almost 4 years ago
accepting feedback specially the image background suggestions
- @Suganya1990Submitted almost 4 years ago
Heyy any suggestions will greatly be appreciated.
@harika09Posted almost 4 years agonot expert here but i suggest to remove the margin from article__title and lower the margin from article__snipet to 10 then lower the padding from article__content to 10 top to fix the display.
1