@Sdann26
Posted
Hi Natalia, the effects you have added and the layout is excellent, only the vertical centering is not a good idea to use margins. You could remove the margins from the card and add to the body instead
body {
display: flex;
flex-direction: column;
justify-content: center;
align-items: center;
min-height: 100vh;
}
This will allow it to be centered on both vertical and horizontal axes.
Finally you have 5 accessibility issues in your frontend mentor report. You can eliminate all of them by changing <div class="container">`` to <main class="container">`, since you should always have a main that contains the main flow of the project in this case is the NFT card.
With these changes you can generate a new report and you will get 0 errors, if you have the same problems in the first challenge with the accessibility issue the same applies.
I hope my comments are helpful, good luck :D
Marked as helpful
@Natalia912
Posted
@Sdann26 Hey! Thank you so much for the feedback, that was extremely helpful! I'll add all those changes now.
