CJuno23• 40
@CJuno23
Submitted
Code-S1
@Code-S1
All comments
- Code-S1• 150
@Code-S1
Posted
check out this article https://fedmentor.dev/posts/font-size-px/ to see the right sizing units and your width is too low on the .result you just need to increase it and just give padding on the .radius
1 - Sainadh7731• 40
@Sainadh7731
Submitted
Code-S1• 150@Code-S1
Posted
its not responsive, and you havent even tried to make it as a responsive page,also there is no box shadow
0 - Pjustine1607• 90
@justine1607
Submitted
What are you most proud of, and what would you do differently next time?
I was amaze by using the css grid, especially the grid-template-area, in where you will just directly add the item without specifying grid column or row anymore, it less code and makes it clean. next time I should do separate the css of the body from the components.
What challenges did you encounter, and how did you overcome them?
the challenges that I encountered, was in the row part of the grid because whenever i add max-height to the 2 cards in the middle there is this empty space that is so annoying, and what i did was I add some grid-template-rows and define its row so i wont get this empty whitespace at the bottom of my 2 middle cards anymore, but I'm open for suggestion of the pro's on how to handle like this situation because for me, on what I just did I think I just add some bandage to that problem that it's really not the solution for that.
What specific areas of your project would you like help with?
the specific areas that i would like help is on the grid-column that has an empty space.
Code-S1• 150@Code-S1
Posted
Overusing <div> tags, known as "divitis," leads to cluttered code, poor semantics, and reduced performance. Instead, use appropriate semantic elements (like <header>, <section>, etc.) to improve readability, accessibility, and SEO. Keep HTML clean and minimal to ensure maintainability, scalability, and better CSS structure.
Marked as helpful
0
