Responsive Order Summary Card with Bootstrap

Community feedback

• 

  Glen Clarence•80

  @Glen-ClarencePosted over 2 years ago

  The Layout looks responsive, and the code is readable! Great Work Keep it up

  Marked as helpful

  0

  ---

  

  Nurul Arifin•310

  @nurularifin83Posted over 2 years ago

  @Glen-Clarence thank u so much Glen, I'm so happy to hear.

  0
• 

  Nurul Arifin•310

  @nurularifin83Posted over 2 years ago

  Hi Grace, thank u so much for your idea.

  For using a different image, for another challenge, I change the image to, I think I like it to change lol.

  As I mentioned before, I am doing everything possible to make design look close to challenging. Right now I'm still learning the Boostrap framework and I'll try Vanilla CSS to. Thank u Grace.

  0
• 

  T

  Grace•31,270

  @grace-snowPosted over 2 years ago

  Hi

  Not sure why you’ve used a different image?

  This isn’t a bad attempt at all but doesn’t look that close to the design yet. Try to get it much much closer if you can.

  Looking at the html…

  • you are using the small tag inappropriately here. Annual plan and the price should be two separate meaningful elements
  • almost never use the br tag. It will be announced to screenreaders as “break” and ruin the flow. There’s no reason to use it these days when you have other html and css tools available
  • why is proceed a button, but change and cancel anchor tags? That may be fine, but what is it you expect to happen on click of each? (As that determines element choice)
  • I strongly recommend against using bootstrap in your other challenges while learning. It is a huge bloated library and will hold back your learning of the underlying css. CSS is changing at pace and you need to be able to leverage modern techniques. Learn vanilla CSS well and you will be able to pick and choose any library you want if you ever want to use one in future, or be able to write bespoke styles with ease

  Good luck

  0