Responsive ( Including SVG ) | Tailwind + Nuxt 3

Community feedback

• 

  Abdul Khaliq 🚀•72,560

  @0xabdulkhaliqPosted over 1 year ago

  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.

  ---

  COMPONENT MEASUREMENTS 📐:

  • The width: 100% property for div element is not necessary. because div is a block level element which will take the full width of the page by default.

  ---

  • So feel free to remove .w-full tailwind utility class from <div class="order-summary ..."> element

  ---

  .

  I hope you find this helpful 😄 Above all, the solution you submitted is great !

  Happy coding!

  Marked as helpful

  1

  ---

  

  dimar hanung•560

  @dimar-hanungPosted over 1 year ago

  @0xAbdulKhalid

  Hi 👋, thanks for reviewing my code!

  div with class order-summary have display:grid, that's mean is not block level element anymore right (?), or maybe i'm wrong.

  but you're right about remove w-full in order-summary, the results are still as expected, i have fixed it

  I use w-full because I'm still confused about the default width characteristic when playing with the display property

  btw thanks for feedback, I got new insight today 🙌

  0