cdnjs / new-website

🖥 cdnjs.com website
https://cdnjs.com
ISC License
451 stars 212 forks source link

[Website] Fix #277: Integrated library name and description in one clickable div #304

Closed mpark86 closed 4 years ago

mpark86 commented 5 years ago
MattIPv4 commented 5 years ago

This feels like a super janky solution -- why not just move where the a element is?

mpark86 commented 5 years ago

Does this look better? I just moved a element to surround the description as well. I also removed all the underlines because it looked a bit messy.

mpark86 commented 5 years ago

I committed another change. I think all the points you mentioned are applied to this one? Could you review? Thanks.

mpark86 commented 5 years ago

All done! This actually looks better.