I mingled with flexbox trying to find perfect solution for card and footer positioning without using position:fixed. I ended up creating an invisible div with h1 tag in it (just to use it because I feel that putting h1 tag as a name in card isn't needed, but accessibility report says otherwise ^^) to be able to use justify-content: space-around property
Javier Aguilar
@aguilarjavierAll comments
- @AbestianSubmitted 9 days agoWhat challenges did you encounter, and how did you overcome them?@aguilarjavierPosted about 15 hours ago
Looks great. Only thing I notice is the main card is slightly above center because of the footer you added.
Overall great job!
1 - @thaliawoodsSubmitted 15 days ago@aguilarjavierPosted 15 days ago
Looks great you go the sizing and overall look down. There is a small on hover effect that is missing from the original design. It's simple it would look something like this.
.title:hover{ color: #F4D04E; cursor: pointer; }
Github link doesn't work at the moment so I can't see how you did your styling.
Overall looks great!
Marked as helpful0 - @SaidosHitsSubmitted about 1 month ago@aguilarjavierPosted 25 days ago
Great job not much to comment on. Sizing of the card was hard to get exact without being given dimensions. Took a look at the code and all I would say is you could use the straight hsl() color codes we were given in the styling guide. Looks like you converted them to hex so it's all the same in the end.
0