Andrea Guarneri• 20
@Andrea-Guarneri
Submitted
Si1entERA
@Si1entERA
All comments
- Si1entERA• 220
@Si1entERA
Posted
Hello @Andrea-Guarneri
Congratulation on the completion of the project.
I'm unable to view your code on GitHub, besides that your project looks good.
0 - MohammadAzmatAli• 50
@MohammadAzmatAli
Submitted
What are you most proud of, and what would you do differently next time?
I am really happy that i implemented grid and flexbox concepts in this project.
What challenges did you encounter, and how did you overcome them?
The challenge was to make the layout responsive in all screen sizes. I managed to do it with the help of CSS grid and flexbox.
What specific areas of your project would you like help with?
Any kind of helpful suggestions...
Si1entERA• 220@Si1entERA
Posted
Hello @MohammadAzmatAli
Congratulations on the completion of your project.
Your project looks amazing and just needs a bit more fine-tuning in your HTML.
You should try "semantic HTML tags" which can help organize your webpage into meaningful sections making it more readable, maintainable and is also good for SEO. You can replace "<div>" with "<section>" or "<article>"
Your project is truly amazing that I was inspired to make some changes to mine.
I hope you find this helpful, good luck on your journey.
Happy Coding.
0 - Kelly Cristina dos Santos• 120
@kellycsantos
Submitted
Si1entERA• 220@Si1entERA
Posted
Hello @kellycsantos
Congratulation on the completion of your project.
Everything looks solid, all that's missing is the hover ability to change the button color.
Happy coding
Marked as helpful
0 - nikol-zimova• 120
@nikol-zimova
Submitted
Si1entERA• 220@Si1entERA
Posted
Hello @nikol-zimova
Congratulations on the completion of your project.
Your project looks solid and all that's needed is some fine-tuning.
You should try "semantic HTML tags" which can help organize your webpage into meaningful sections making it more readable, maintainable whiles giving you more control on the layout and is also good for SEO. You can replace "<div>" with "<section>" or "<article>"
The way you centered your div is solid, but it's always good to have more than one option to center a div. Below uses flex.
body{ display: flex; justify-content: center; align-item: center; min-height: 100vh; }
I hope you find this helpful, good luck on your journey.
Happy Coding.
Marked as helpful
1 - Derek Walton• 20
@Derek-Walton
Submitted
What are you most proud of, and what would you do differently next time?
seemed quite similiar to the design
What challenges did you encounter, and how did you overcome them?
went quite smoothly
What specific areas of your project would you like help with?
any help would be appreciated :)
Si1entERA• 220@Si1entERA
Posted
Hello @Derek-Walton.
Congratulations on the completion of your project.
I'm still new so my ability to give helpful solution and advice is limited, but from what I can see, you have a solid project. Try reducing the spacing between "<h1>Jessica Randall</h1> and <h2>London, United Kingdom</h2>" This can be a challenge since changing the "gap." value can affect the appearance of the project.
Also look into using "section tags" within your main tags. This can help organize the content into meaningful sections making it more readable, maintainable and can also give you more control on the layout.
I hope you find this helpful.
Happy coding
Marked as helpful
1 - joacomenda• 150
@joacomenda
Submitted
What are you most proud of, and what would you do differently next time?
I'm most proud of how it ended up looking in desktop, I would try and intregrate more media queries so I can make it more responsive to mobile devices
What challenges did you encounter, and how did you overcome them?
I found difficult to center the container of all the card content to the center of the page, I contained everything into a section element, then I gave it height. Then I styled it using CSS, adding a flexbox to align it horizontally and vertically.
What specific areas of your project would you like help with?
I'm still finding another to put the container in the center of the page, if you have another way, plase share it.
Also, what is the best practice to make a responsive design, to make it look good on mobile
Si1entERA• 220@Si1entERA
Posted
Hello @joacomenda.
Congratulations on the completion of your project.
I'm still new and I'm not at the level of the other developer here but I'll pass on some advice I recently received from @danielmrz-dev.
"Try using<h1> to <h6> tags for headings instead of<p> since they are used for paragraphs. When it comes to headings. This can improve the structure of your HTML and make it look cleaner."
This is one of my mistakes I've made a lot and I hope you find this helpful. Happy coding.
Marked as helpful
2 - Q_421• 30
@ahmaddanu421
Submitted
What are you most proud of, and what would you do differently next time?
I still don't know, i just doing trial and error. This is my first try. I think i should learn git.
What challenges did you encounter, and how did you overcome them?
CSS, i can't understand how CSS layouting works, it's kind of hacky. I just try random property and value which i think it should work, but sometimes it doesn't work.
What specific areas of your project would you like help with?
What should i do to improve my code?
Si1entERA• 220@Si1entERA
Posted
This is my first comment, but I hope this helps, Try using the link tags below and place it in the head tag.
"<link rel="preconnect" href="https://fonts.googleapis.com">
<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin> <link href="https://fonts.googleapis.com/css2?family=Outfit:[email protected]&display=swap" rel="stylesheet">"I'm not familiar with "@import url("https://fonts.google.com/specimen/Outfit");" since I'm still learning.
Marked as helpful
0
