Reuben👾• 560
@Rubylenshy
Posted
Hi, @Ineke84 👋. Congratulations on completing this challenge, you did a nice job. You could try these few suggestions out on the accessibility--the best practice like you said:
- The <div class="container-main"></div> should be wrapped in a <main></main> tag, for easy recognition by any web browser, it stands out as the main content of the web page or just rename the 'div' to 'main'
- How about you make <div class="attribution"></div> a footer instead, seems cool and more structural.
- You could also add an 'alt' attribute to the image. Kudos 👌.
- And an easy way to position your card at the center of the page is: .conatainer-main {position: absolute; top: 50%; left: 50%; transform: translate(-50%, -50%)
I hope this helps :)
Keep Coding @Ineke84
Marked as helpful
0
