AlexCristea2007• 380
@AlexCristea2007
Submitted
Faisal Baig
@Faisalbaig1998
All comments
- Faisal Baig• 90
@Faisalbaig1998
Posted
The image could have been better, and the paragraph spacing should have been equal.
0 - Faisal Baig• 90
@Faisalbaig1998
Posted
The container width should be smaller.
1 - Ki'ara Colbert• 120
@kiaraaa123
Submitted
Faisal Baig• 90@Faisalbaig1998
Posted
The layout remains the same, but the card shadows should be more blurred. Additionally, the font color is too dark. Please pay attention to these details.
0 - Rowan Pereira• 330
@rowanrooster
Submitted
Faisal Baig• 90@Faisalbaig1998
Posted
None
0 - BhargavMarala• 80
@BhargavMarala
Submitted
Faisal Baig• 90@Faisalbaig1998
Posted
None
0 - errbrokko• 80
@errbrokko
Submitted
Faisal Baig• 90@Faisalbaig1998
Posted
none
0 - oluwatoyosi• 20
@wetwan
Submitted
What are you most proud of, and what would you do differently next time?
this is the first time i'm trying responsive screen
What challenges did you encounter, and how did you overcome them?
getting the box shadow right
What specific areas of your project would you like help with?
i am trying for now
Faisal Baig• 90@Faisalbaig1998
Posted
Nice
0 @arkus7
Submitted
What are you most proud of, and what would you do differently next time?
I'm most proud that I managed to finish this challenge :) Next time, I might separate CSS from the index.html file and maybe use some variables (or even SCSS) for styling.
What challenges did you encounter, and how did you overcome them?
I had a problem with centring the card both vertically and horizontally. I found some articles about using
flexto do this, but the default margins and padding messed it up a bit (creating additional space so that scrollbars were active). I've used a "hack" with*to reset those, but I'm unsure if this is the right approach. I've also had an issue with properly sizing the text elements on the card - the style guide said the paragraph size should be 15px, but there was no mention of the size for the header. I had to download the Figma file to check the proper size so I could mirror the design as closely as possible.What specific areas of your project would you like help with?
- I'm not sure if the HTML structure I came up with is the right approach. I tried not to overuse
divelements, I reached for tags likemainandsectionto give it more semantics although I worry that this is overkill for the example in the challenge. Any feedback is appreciated. - I had a problem with centring things and I've used
flexto solve it - what is the best approach here? I've usedheight: 100vhon the container to fill the whole viewport so it is also centred vertically - is this how it's supposed to be done? - Is the naming of the classes in the CSS fine? I've always struggled with naming them and I went for naming by the "component" they describe.
Faisal Baig• 90@Faisalbaig1998
Posted
Nothing
0- I'm not sure if the HTML structure I came up with is the right approach. I tried not to overuse
