@asdiAdi
@benhyh
Submitted
What specific areas of your project would you like help with?
The link only works when the user cursor is directly on the anchor tags of the link. In other words, if users hover somewhere else in the social links container, the link will not work.
@asdiAdi
Posted
Really well done!
@A-Realist
Submitted
What are you most proud of, and what would you do differently next time?
After a long break, I'm finally starting to learn Web Dev and I'm quite happy with the result. I'll try to make other projects mobile-responsive.
What specific areas of your project would you like help with?
There are mainly 2 problems that I was unable to solve:
The QR code image has some space from the top which I'm unable to get rid of. I've a padding of 25px which gave a nice margin to the entire card but the top has a little more space than required.
The text inside tag: is not aligned like it is in the original design. I could have used 3 tags but maybe it wasn't the best practice.
@asdiAdi
Posted
I think it's better to put some alt text on the image. Also make the card fixed width so it won't shrink on smaller screen sizes.
@ValarieOyieke
Submitted
I wasn't able to align the div with the prices at the center with display: flex; align-items: center; so I used align-items: baseline instead.
Any help on what might have caused it not to work will be highly appreciated.
@asdiAdi
Posted
Hi Valarie Oyieke, it seems that the p margins inside the .price class are the root of the problem.
.price > p selector and add margin: 0 property inside itmargin-top: 32px property inside the .price classalign-items property into center inside the .price classI hope this helps you and Good luck with future challenges!
Marked as helpful