P
@YonerfySubmitted 14 days ago
P
JeronimoCardu
@JeronimoCarduAll comments
- P@JeronimoCarduPosted 7 days ago
You can create a download button component to avoid repeating the same button in your code. Since it's in a landing page, it might be used multiple times. And the color of the version in the footer is wrong.
But is excelent!
0 - @Akiz-IvanovSubmitted 12 days agoWhat are you most proud of, and what would you do differently next time?
What i would've done differently is made it without React if it was a static page without the need of adding more cards in the future. I'm glad that i got more hands-on practice with React, but it certainly took more time than necessary.
What challenges did you encounter, and how did you overcome them?The hardest challenge was making the layout responsive. I over-engineered it a bit, mainly to allow for the option of adding more cards in the future.
What specific areas of your project would you like help with?Without the Figma design, I might have missed some colors or the box-shadow.
P@JeronimoCarduPosted 12 days agoHey, if you don't have Pro and are having trouble with colors, you can install the ColorZilla extension in Chrome. This tool allows you to pick colors in RGB, HSL, and HEX formats. Just open the free design images in your browser and grab the colors!
Marked as helpful1 - @EdebaliErmanSubmitted over 1 year ago
- @valen-webdSubmitted 16 days agoWhat are you most proud of, and what would you do differently next time?
I'm really happy with how this turned out! 😊️
This feels like my cleanest, most professional-looking solution/demo yet.
What specific areas of your project would you like help with?If anyone knows if there is a better way to horizontally center icon and text in the
<button>element, please let me know!I used
display: flex;withalign-items: center;. It works, but it feels unnecessary to change thedisplayproperty to align the items? - @thejas2246Submitted 19 days agoP@JeronimoCarduPosted 18 days ago
<div class="table"> <div>Calories</div> <div>277kcal</div> <div>Carbs</div> <div>0g</div> <div>Protein</div> <div>20g</div> <div>Fat</div> <div>22g</div> </div> </section>You will can replace this for a table. but is great!
Marked as helpful0 - @Winnie139Submitted 19 days ago
- @Ian-CardosoSubmitted 19 days agoP@JeronimoCarduPosted 19 days ago
Me parece que esta bien, aunque el HTML podría ser mas semántico. Saludos!
0 - @soseenSubmitted 20 days agoWhat are you most proud of, and what would you do differently next time?
I would pay more attention to the text sizing.
What challenges did you encounter, and how did you overcome them?To nail the text wrapping.
What specific areas of your project would you like help with?None
P@JeronimoCarduPosted 19 days agoEsta bien. Lo haria un poco mas semantico, cambiando el
-
<div class="card">por<article class="card"> -
<div class="text-container">por<div class="text-container"> -
El
<h2>lo cambiaria por un<p>
El resto me parece perfecto, saludos!
0 -
