Olanrewaju
@LANRIE-DEVAll comments
- @AdkasySubmitted 7 months agoWhat are you most proud of, and what would you do differently next time?@LANRIE-DEVPosted 7 months ago
@Aditya, I would like to commend you on a job well done, but if you wouldn't mind, I would like to put down some suggestions.
-
Firstly, you can do well to increase the font size of your headings (h1, h2) and also other elements.
-
Also, you can add some margin to the top, so it is matched with the design.
Again, thumbs up on the job. 👍👍👍😉
Marked as helpful0 -
- @mrbrave7Submitted 8 months agoWhat are you most proud of, and what would you do differently next time?
I Mostly Proud That I Can Create A Full Creative Pages And Designs Using Html And Css Without Any Frameworks
What challenges did you encounter, and how did you overcome them?It Was easy But I Could Not Get that What Could Be The Width Of main Card Container
What specific areas of your project would you like help with?Every Little Code Will Help You
- @MaxCodeCraftSubmitted 8 months agoWhat are you most proud of, and what would you do differently next time?
This is my first responsive project, with more attention to the margins and paddings than the previous one ! I've really enjoyed this one as it was more complex and made me Google a thing or two so I learnt a lot with it.
What challenges did you encounter, and how did you overcome them?Here is what I learned during this project:
- Using @font-face to import custom fonts
- Better understanding of the Flexbox model
- How to make a container responsive
I didn't manage to have the left and right margins of 24px when viewed on mobile, if I did put margin: 0 1.5rem, i had the margin on the left side only and the right side was cropped, any ideas ? Thank you
@LANRIE-DEVPosted 8 months agoI would like to commend you on a job well done. 👍👍
I would love to make a few suggestions.
-
⭐️ Great job using semantic HTML! Such widgets as cards are more suited to be constructed with the <article> element, which encapsulates reusable, self-contained content.
-
You didn't include the transition property for the active state for the heading (HTML and CSS foundations)
This is mine 👇👇:
<h1 class="main heading">HTML & CSS foundations</h1>
The styling below 👇👇:
`. main heading { font-size: 1.5rem; font-weight: 800; display: inline-block; margin: 1.1rem 0; transition: color 0.2s ease; }
. main heading: hover { color: hsl(47, 88%, 63%); cursor: pointer; }`
Again, well done and thumbs up. 👍
Marked as helpful1 - @luminous42Submitted 8 months ago