Frontend Mentor | Responsive ( Including SVG ) | Tailwind + Nuxt 3 coding challenge solution

Community feedback

@0xabdulkhaliq

Posted 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

@dimar-hanung

Posted 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 🙌

Responsive ( Including SVG ) | Tailwind + Nuxt 3