I had difficulty translating some of the design aspects in Figma to CSS. For example, finding the border radius of the QR code image was difficult. I had to try clicking on different elements in the design before I got to a view that showed me the necessary info.
Karim Essam
@KaEssamAll comments
- @sbappanSubmitted 5 months agoWhat challenges did you encounter, and how did you overcome them?@KaEssamPosted 5 months ago
This solution has a nice layout, It does not really differ from the design.
0 - @GehadazzamSubmitted 5 months ago@KaEssamPosted 5 months ago
This solution has a nice layout, It does not really differ from the design. you need only to change the padding in the white section
Marked as helpful0 - @ezequielastradaSubmitted 5 months ago@KaEssamPosted 5 months ago
This solution has a nice layout, It does not really differ from the design.
0 - @hichamweblogSubmitted 5 months agoWhat are you most proud of, and what would you do differently next time?
I am most proud of the responsive design of the landing page
What challenges did you encounter, and how did you overcome them?One of the challenges I encountered was managing the hero section's design, which features a single image on mobile and tablet views but two images on the desktop view. I believe this could be handled more efficiently.
What specific areas of your project would you like help with?I would like help with improving the accessibility and writing cleaner, more DRY CSS.
@KaEssamPosted 5 months agoThis solution has a nice layout, It does not really differ from the design.
keep building !
1 - @PrestonLTaylorSubmitted 5 months agoWhat specific areas of your project would you like help with?
I would like feedback on the implementation of my CSS styles.
@KaEssamPosted 5 months agoYour solution looks good. add padding for white card. Overall, it's great!
0 - @JoelJohsSubmitted 5 months ago@KaEssamPosted 5 months ago
Your solution looks good. Please path for images, change size for card make it small. Overall, it's great!
1 - @AmnaAs1fSubmitted 6 months ago@KaEssamPosted 5 months ago
Your solution looks good. Please change the body background and add padding to the right section. Overall, it's great!
0 - @FunTomDevSubmitted 6 months agoWhat are you most proud of, and what would you do differently next time?
I am most proud of learning how to work with scrollable pages and how to make them responsive. There is nothing I would do differently next time.
What challenges did you encounter, and how did you overcome them?The greatest challenge was to find the best way of aligning the main box in the middle of the page without changing the behavior. To do that I simply added another div container and instead of setting my to display flex, I set up my element to display: flex and it worked fine.
What specific areas of your project would you like help with?There are none.
@KaEssamPosted 5 months agoYour solution looks good. You need to change the background color for preparation time. Overall, it's perfect.
0 - @ajmilesomniSubmitted 5 months ago@KaEssamPosted 5 months ago
Your solution looks good. You need to adjust the width for the links. Overall, it's perfect.
Marked as helpful0 - @denisson999Submitted 5 months ago@KaEssamPosted 5 months ago
Your solution looks good. You need to adjust the image for the author and add padding at the bottom. Overall, it's perfect.
0 - @kaustubhyaSubmitted 5 months agoWhat are you most proud of, and what would you do differently next time?
Proud - Tried to make responsive
TODO - Try to include this knowledge and apply on other projects
@KaEssamPosted 5 months agoyour solution looks great and matches the design well. However, you included the attribution within the card div. Please move it outside the card div.
0