Order Summary Component using HTML, CSS

Community feedback

• 

  Arshad Ali Kaldane•650

  @IamArshadAliPosted 12 months ago

  Hello There! 👋

  Congratulations on completing this challenge. 🎉

  I've some suggestions that might interest you. 💡

  • use overflow: hidden; on your .card, it will take care of the overflowing image, and the specified border-radius will be applied to the card.

  • you can add a screenshot into the readme like this ![alt text](./src/images/screenshot.jpg) or this <img src"./src/images/screenshot.jpg" />

  Hope this helps you. 👍

  Happy Coding 🤓

  Marked as helpful

  0

  ---

  

  Malgorzata Stano•70

  @gosiastPosted 12 months ago

  Hi! @IamArshadAli :) Thanks a lot for your help! Fixed now! :)

  Do you know why the screenshot solutions break my "change" link? I checked, and it looks correct on my machine.

  Thanks and have a nice day!

  0

  

  Arshad Ali Kaldane•650

  @IamArshadAliPosted 12 months ago

  Greetings @gosiast The challenge's screenshot provided to us are captured at a width of 1440px for desktop and 375px for mobile screens. So, it's important to pay special attention to these screen sizes.

  You might have a display with a screen resolution of 1920x1080 or something else, which is why the design looks perfect on your machine and breaks in the screenshot, as the screenshot mechanism looks for our design at 1440px width.

  You might want to consider simulating your screen using the developer tools of your browser. Most of the browsers follow the same path to access the developer tools i.e. options > more tools > developer tools or you can simply use the following key combination Ctrl (or Cmd) + Shift + I.

  Then set the dimensions of the screen to the desired size and you are good to go.

  Hope this helps. Thank you🤓

  Marked as helpful

  0
• 

  JimTK16•370

  @JimTK16Posted 12 months ago

  Hi, i think you only need to set border radius for the top container, in this case is the order--wrapper. You can do something like this {border-radius: 2rem; overflow: hidden}. The overflow: hidden is to keep the image from overflowing the container, and help to reveal the top corners. To insert images to readme file you could do a quick search about insert images in markdown.

  Hope it helps.

  Jim

  Marked as helpful

  0

  ---

  

  Malgorzata Stano•70

  @gosiastPosted 12 months ago

  Hi! @JimTK16 :) Thanks a lot for your help! Fixed now! :)

  Do you know why the screenshot solutions break my "change" link? I checked, and it looks correct on my machine.

  Thanks and have a nice day!

  0

  

  JimTK16•370

  @JimTK16Posted 12 months ago

  Hi @gosiast, Your site looks nice on big screens. However, you need to also design for small screens as well. I suspect that they captured your solution screen when it's on a small screen, and it makes the 'Change' wrap to the next line due to lack of space. You can use media queries to adjust the layout and size of different elements on different screen sizes. Here a link to my code for an example : https://github.com/JimTK16/order-summary-component

  Marked as helpful

  0