@mjbaga
Posted
Hi, @tab21. Nice job on the card. It looks great.
I've checked the code and see that you're using a background image. I also suggest try doing alternate one where you're using an image tag. This might not be useful now but typically on the job, the image tags are the way to go because:
- Screen readers (assistive technology) won't be able to read background images on CSS. They can read image tags and their alt attributes.
- Sometimes when working with backend and the image source comes from them, they can't place the image unless they use a style tag and some policies like CSP prevent inline styles.
Otherwise, solution is great. Hope this helps. Happy coding.
@tab21
Posted
@mjbaga hi thanks for telling the industry practice. Will incorporate this in the future code ;)