Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Order summary card design with BEM/flex/CSS3

Parviz Parastarโ€ข 80

@Parviz-Parastar

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello to all friends who are active on this site.๐Ÿ‘‹

In this exercise, I tried to write the elements of each section according to the BEM method completely and accurately and have a clean code.

To build a better world, I will be very happy to read your comments and help each other to progress.๐ŸŽ‰

Community feedback

Abdul Khaliq ๐Ÿš€โ€ข 72,660

@0xabdulkhaliq

Posted

Hello there ๐Ÿ‘‹. Congratulations on successfully completing the challenge! ๐ŸŽ‰

  • I have other recommendations regarding your code that I believe will be of great interest to you.

BACKGROUND iMAGE ๐Ÿ“ธ:

  • The background-image has not been properly set yet, they image seems to be disorted while the user tries to zoom-in or zoom-out the window. So let me explain How you can easily apply the background color along with the svg they provided.
  • Add the following style rule to your css, and then experience the changes
body {
background: url(./images/pattern-background-desktop.svg), #E1E9Ff;
background-repeat: no-repeat, no-repeat;
background-size: contain;
}
  • Tip, Don't forget to generate a new screenshot after editing the css file

.

I hope you find this helpful ๐Ÿ˜„ Above all, the solution you submitted is great !

Happy coding!

Marked as helpful

1

Parviz Parastarโ€ข 80

@Parviz-Parastar

Posted

@0xAbdulKhalid

Hello to the professional front end developer Mr. Abdul Khalid.๐Ÿ‘‹

I am very glad that you have reviewed my solution and sent me a valuable suggestion. You are one of the always active and professional members of the frontend mentor site and I am very happy that I can use your experiences to improve.

Yes that's right. I code background-size: contain; I used it for the elements and boxes inside the Body, but I didn't know that I should apply this to the background as well.

And I had a small question that we must apply the second no-repeat which is for the background-color? background-repeat: no-repeat, no-repeat; Because I don't see any effect in writing or not writing for the background color. Is it necessary to write? I am waiting for your answer.

Thank you very much.๐Ÿ™๐Ÿงก

0
Luka Glontiโ€ข 3,440

@lack21

Posted

Great job ๐Ÿ‘, but I have a suggestion!

  • Add background-size: contain to the body, like this image will take the full width!

Marked as helpful

1

Parviz Parastarโ€ข 80

@Parviz-Parastar

Posted

@lack21 Hi dear friend . Lucas Glonti.๐Ÿ‘‹

Yes that's right. I code background-size: contain; I used it for the elements and boxes inside the Body, but I didn't know that I should apply this to the background as well. I am very grateful to you.๐Ÿ™

1
tahaโ€ข 100

@M-Taha-Amin

Posted

Looks great, Cheers mate ๐Ÿ™Œ

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord