@jenmurph4610
Posted
Hi @underworl, nice work!
I found the use of the flex display property helpful in my version of this project, thanks to feedback I received. It made it so much easier to center the items within the body, but also center the items within the 'card' itself.
You could get rid of the divs within .container and leave just the three elements (img, p, p) within one div themselves. Arranging them with flex within that will then be easier. I was able to eliminate a lot of margin properties and such from my CSS when I did this.
Your first <p> should probably be a heading like <h1> too, I think that will eliminate one of your errors in the accessibility report, but with that you could also simplify more and eliminate the classes from all three and just target them with the selectors since there would be only one of each type.
I tend to work on the same before I work positioning as well. It seems to work well for me too but curious to see what others might input on that.
@Pinedevs
Posted
@jenmurph4610 Thank you for the feedback. I'll try using less unnecessary divs and work on my use of flexbox.
It's good to know that the way I tend to work also can work well.