Proud that I have actually managed to come up with a responsive solution.
What challenges did you encounter, and how did you overcome them?Idk this one went really well.
What specific areas of your project would you like help with?Actual structure
I would like help on how to use bootstrap grid to achieve a similar result.
Any form of help to get better will be much appreciated.
I will need help probably on refactoring my code.
Proud that I have actually managed to come up with a responsive solution.
What challenges did you encounter, and how did you overcome them?Idk this one went really well.
What specific areas of your project would you like help with?Actual structure
Your design is identical to the actual design. Good job
Positioning the cards in this way, and making the layout responsive huge pain in the ass.
What specific areas of your project would you like help with?When it falls to tablet breakpoint I wanted to make it 2 column but I couldn't manage that because of the layout.
I think your design is perfect, it looks like the real thing and it is mobile friendly. The only thing that I would say which is not really a problem is the font-weight of your h1, I think you should reduce it to 200. But that isn't really a problem.
Rodri, your site looks very nice. This is just the one thing I will like to add. The "perfume" text you can add this as the style for this specific text letter-spacing: 0.25rem; text-transform: uppercase;
Hey Rodri, your site is well structured and the design is alright. It just needs a little more style. For the fonts, I think you can add this link for the font in the head section of your html code.
<link rel="preconnect" href="https://fonts.googleapis.com" /> <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin /> <link href="https://fonts.googleapis.com/css2?family=Outfit:wght@100..900&family=Young+Serif&display=swap" rel="stylesheet" /> Then you can style the font in your CSS file like the font-family: Outfit; (For the body) font-weight: 400; (For the heading elements)For the table, you can use this for styling tr, td { padding: 5px 35px; } This is just to make it look almost like the real thing. And in my opinion I think that any style you want for your website, you should put in your CSS file. This will make your code less clustered.
I think your code is really good and I picked up two things from your solution that helped me.
I am proud of successfully utilizing flexbox in this challenge and completing it with a high level of energy.
What challenges did you encounter, and how did you overcome them?Upon initially downloading the assets, I encountered difficulty in creating responsive card. As a result, I took the initiative to educate myself on Flexbox before proceeding with this challenge.
I really like your design, it is so close to the original design. The only thing that I see is that the "learning" which is in an anchor<a> tag in my opinion, though I might be wrong, is not a button. I think you can make it a header<h1-h6> or paragraph<p> element. Then style it like this background-color: hsl(47, 88%, 63%); display: inline; padding: 5px; border-radius: 3px;