@Nurio34
ReeperC3
@ReeperC3All comments
- @ReeperC3
Really concise code and the solution looks great over all. Nice work!
- P@Henolivares@ReeperC3
I can't view the preview or the code but the screenshot looks really accurate and nice all around.
- @dimasrdevWhat are you most proud of, and what would you do differently next time?
It's been a while since I've code, so I'm just picking up again now. I'm proud to say that I am still able to do some HTML and Tailwind CSS. I hope I can be a real frontend developer one day.
What challenges did you encounter, and how did you overcome them?I encountered some challenges on the table part. But, some Google searches helped me.
What specific areas of your project would you like help with?What can be done to clean my code? Is it already following best practices? I would love to know how a seasoned developer would code it. Many thanks!
@ReeperC3Mostly looking really nice and accurate. Nice semantic HTML. Can't really fault anything here
- @kusumanth123@ReeperC3
The spacing between the links is way too much and I believe the font style is incorrect. The image also doesn't seem to load and there is no hover state for the links. There is no use of semantic HTML so consider using
<section>or<article>tags instead. I would also suggest putting the CSS code into a separate styles.css file. - @OneManBanned@ReeperC3
The final result looks really accurate and nice. Great use of semantic HTML and the separation of mobile and desktop sizing is really nice. CSS is very well structured.
- @ayeryn@ReeperC3
The sizing of the text is not absolutely perfect but it looks fine nonetheless. Your code is very well structured and the whole thing looks fine. If I had one thing to change I'd fix the positioning of the text so it lines up with the original image a bit nicer.
