@micfrazer
@Ridwan10000
Submitted
What are you most proud of, and what would you do differently next time?
I am proud of completing this project. I tried to make it as close as I could
What challenges did you encounter, and how did you overcome them?
I encountered to fit the page in mobile devices
What specific areas of your project would you like help with?
@micfrazer
Posted
The page looks great! Consider a max-width property to allow for the image and main portion of the body to adjust with smaller windows, though the sizing is very consistent between devices.
The code is very clear and well-organized. You might use the following to change the colors of bullets and numbers in itemized lists: li::marker { color: rgb(136, 70, 56); }
The table is a bit inconsistent in its layout, though it's clear overall. Padding, width, and text-align are what I found incredibly helpful here. Overall, incredible work!
@elekviktor32
Submitted
@micfrazer
Posted
Visually, this looks great! I might suggest a little more space between the buttons with a slightly larger gap for ease of user experience/usage (were this a full app).
Nice use of rem measurements to maintain consistency. The inclusion of button interactivity with pseudoclasses is a nice touch as well!
Overall, the HTML and CSS look very well organized and clear. Great work!
@Kevinjoan07
Submitted
What are you most proud of, and what would you do differently next time?
learn by using the call to protocol or external apis and include them to my components
What challenges did you encounter, and how did you overcome them?
How to follow all the instructions described in the Guide
What specific areas of your project would you like help with?
I would like full feedback on what I can improve.
@micfrazer
Posted
The result looks excellent! The semantic tags in the HTML look great. The CSS is likewise very clear (and actually gave me some insight into flex/grid, so thank you).
The border is slightly thicker than the original, but that's stylistic preference (and I prefer the 2px you used, actually). Remove the extra text from the footer (namely the blog material), but that's only a minor textual redundancy at most. Well done!
Marked as helpful
@AntonB12
Submitted
What are you most proud of, and what would you do differently next time?
estoy orgulloso de haber completado mi primer ejercicio desde que me recomendaron esta pagina, lo que haria diferente tal vez seria buscar una forma en la cual pueda usar menos codigo ya teniendo entendimiento de que es lo que estoy haciendo
What challenges did you encounter, and how did you overcome them?
encontre desafios como el de darle sus espacios a cada elemento dentro de la tarjeta y aprender a dimensionar estos para que todo sevea estetico y bien
What specific areas of your project would you like help with?
me gustaria que me dieran ayuda en general sobre todo el proyecto, seria muy bueno recibir ayuda de todo tipo y consejos para mejorar mi codigo
@micfrazer
Posted
CSS is very clear! Semantic tags in the HTML could be helpful (e.g. <main>, <footer>); this helps significantly in terms of accessibility. The overall layout looks good in multiple sizes and seems to be responsive; you might consider scaling the size of the object itself as well (though it reads pretty well overall). Consider padding elements for consistent layout, especially in the text. Good work!
