What are you most proud of, and what would you do differently next time?
I am very proud due to the fact that I focused on not looking at my notes and just trying to code this as if I was working on a project for a client on a deadline. I tried to use the documentation for many of the aspects that I was unsure of and played around with styling for a little before I figured out what I needed to do to get the display that I was looking for.
What challenges did you encounter, and how did you overcome them?
This biggest challenge I had was styling the @media query although I know it is not perfect, I am happy with the results. I used the documentation on media query to do the best that I can, and I am very happy with how it turned out in the end, especially in the sense of scalability.
What specific areas of your project would you like help with?
The media query if someone wants to explain what I did wrong or what I can do better next time, I am all ears.
In your HTML file, as best practice, all the code should be inside the <body> like this:
<!DOCTYPE html>
<html>
<head>
</head>
<body>
your code should be written here
</body>
</html>
then add a body selector in your CSS file and apply the background-color: #D5E1EF; inside instead of html, which is useless in our context.
Also, to center your card display:flex; justify-content:center; align-items: center in the body selector.
-Change the color of the h1 to black to match the design
This is just the beginning of your awesome journey as a front-end developer Keep up the good work.
I hope this was helpful, and if you need any help, don't hesitate to contact me here.
Hi, you did well. You are a few steps away from matching the design.
Change the body background to hsl(220, 15%, 55%).
Add justify-content:center; align-items: center; to your body selector in the CSS file to center your qr-component in the middle of the screen horizontally and vertically.
Remove the background color from the card selector so it remains white by default.
Set the border radius to 5% to match the design.
I can't see why divide the paragraph on two p tags, put the inside one p tag
Instead of using the <center> tag add text-align: center; to the card selector in css file, same the for the h2 tag.
This is just the beginning of your awesome journey as a front-end developer Keep up the good work.
I hope this was helpful, and if you need any help, don't hesitate to contact me here.
Hi Alejandro, You did well with the design; it looks good.
For mobile, try to place share-icon and share_div in the same div and hide the share_div initially. Once the user clicks the share_icon, hide the user_div and display the share_div, displaying them on top of each other is not a good practice.
Add the else case once the user want to hide the share options, it's not included in your code just by adding an else statement that undoes what you changed in the if statement .
Add some semantic HTML to make your code accessible to screen readers.
Writing CSS code in a separate file will make it more organized and easier to manage and maintain.
For the nutrition table, I encountered the same problem and overcame it by using three different div with the same class and styling them to look like the design you can check my work here.
For accessibility, write something that describes the image in the alt attribute.