Design comparison
Solution retrospective
I almost gave up the moment I faced the first problem, which was linking my github to Vscode. I kept trying different things, reading different articles but I was still getting error messages. I am happy I didn't leave it. As simple as it looks I spent over 5 hours understanding working on the Css. Next time I would be more calm when it comes to reading instructions.
What challenges did you encounter, and how did you overcome them?Connecting Git, Firguring out how to make the page static, aliging the content to the middle, and lastly I struggled with making the page go live
I kept asking chatgpt questions, I didn't give up basically
Community feedback
- @DylandeBruijnPosted 4 months ago
@ZoeInMyDna-code
Hiya! 👋
Congratulations on your solution, it looks very close to the design! I can tell you put a lot of effort into it. Props that you didn't gave up and pushed through!
Things you could improve ✍️
-
I suggest adding a bit of
padding
to yourbody
element so the card has some space around it on smaller viewports. -
You could add a
min-height: 100vh
to yourbody
element so it takes up the full height of the viewport while still being able to grow when the content inside it grows. -
Try experimenting with CSS variables, they help you make your CSS values more reusable across your code.
-
I suggest using clear descriptive CSS classes like
.card
,.card-title
and.card-description
. -
Try using semantic HTML elements like
main
andarticle
. -
Try using using relative CSS units like
rem
andem
they make your layout more adaptable. -
Would change your image CSS:
.image { display: block; border-radius: 15px; max-width: 100%; }
So this means that your image will take up the full width of it's container but never will be larger than it's container. I always put this as a base on my images, it scales really well.
-
Try adding the right
font-family
to your solution. -
I recommend not using ChatGPT too much when you are just starting out because you can't tell when it's giving you wrong information. Try to master the fundamentals first of HTML and CSS, really try to understand what goes on under the hood. Look into the box-model, and the cascade for example.
I hope you find my feedback helpful! 🌟
Let me know if you have more questions and I'll do my best to answer them. 🙋♂️
Happy coding! 😎
0@ZoeInMyDna-codePosted 4 months ago@DylandeBruijn
You really took out time to access the code and the design and brought out constructive feed backs for me.
I would do well to take note of the correctioms and see if I can rewrite some of the codes.
What do you suggest I do when I need a tag and I don't know the name where can I find what to do
0@DylandeBruijnPosted 4 months ago0 -
- @MoamenAbdelrahman7Posted 4 months ago
paragraph font size should be smaller
0@ZoeInMyDna-codePosted 4 months ago@MoamenAbdelrahman7
About how many px I would like to heed the correction
0@ZoeInMyDna-codePosted 4 months ago@MoamenAbdelrahman7
About how many px I would like to heed the correction
0@MoamenAbdelrahman7Posted 4 months ago@ZoeInMyDna-code make it about 12px or 13px
0 - @bassdesignsPosted 4 months ago
Kudos for not giving up, well done and continue to improve
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord