Responsive Order summary component using flex CSS, hover and pointer

Community feedback

• 

  Anubliss•460

  @Anubliss-0Posted about 3 years ago

  Hi Tunde,

  Sadly I cant see your website!

  On the project settings on github, you can make the repo public to unlock the option to host the page for others to view.

  having a quick glance at your code, the CSS looks great! The use of class and selectors is well done.

  I notice the containers are set to flex, you could also considering using media queries to further change the layout of your page for smaller devices.

  One tiny point to mention! The w3c standard for indentation is 2 spaces. I noticed your nested elements have 4 spaces of whitespace at the start.

  Otherwise it looks clean, great job completing the task!!

  Anubliss_

  Marked as helpful

  0

  ---

  

  Adekoya Tunde•10

  @TundeAdekoyaPosted about 3 years ago

  @Anubliss-0 thanks a lot on the feedback. Here's the link if you still wanna check it out https://frontend-challenge.tundeadekoya.repl.co

  1

  

  Anubliss•460

  @Anubliss-0Posted about 3 years ago

  @TundeAdekoya

  The site looks great! Thanks for updating your comment :) The only thing I noticed is that your body has a fixed width, Looks fine on my laptop, but when plugged into a larger display I noticed the box moves away from the center.

  I'm still a learner myself, but perhaps using a value like % or vw instead of px would alleviate the issue.

  Anubliss_

  Marked as helpful

  0
• 

  Rahul.K•445

  @rahulkhimsuriyaPosted about 3 years ago

  Hi There 👋

  Congress on completing your 1st challenge.

  when you publicly open your code then provide preview link of that so other people can see that and learn from that too.

  you can deploy your code on vercel or netlify free of cost.

  keep coding...

  Marked as helpful

  0

  ---

  

  Adekoya Tunde•10

  @TundeAdekoyaPosted about 3 years ago

  @rahulkhimsuriya thanks, will do

  0