@fa7eh
Posted
Great work 👍🏽
Few suggestions
You don't need to specify display: block
everywhere. Remove it from all elements except img and you won't see any difference.
I see you are specifying margin: 0 auto
in lot of places. I understand you are trying to center things but as Abdul mentioned in his comment, there are better tools to center elements in css.
I see you specified padding
for img and p element to keep them inside the wrapper. The best solution is to just give padding to wrapper itself and all elements will stay inside that padding.
.qr-wrap {
padding: 15px;
}
I hope you find this useful. Best of luck.
Marked as helpful