@FocusCookieSubmitted over 3 years ago
trinity-senpai
@trinitysenpaiAll comments
- @trinitysenpaiPosted over 3 years ago
wow! I like how you used css variables, makes it easier to maintain and keep track of styles. I'm actually learning some new stuff from your css code, like the
gap
property :O and how you used thebackground-position
property.- The middle review card looks like it needs some padding on the bottom when the width is on
950px
- not sure about the width of the cards when on tablet device,
max-width
might do the trick.
0 - The middle review card looks like it needs some padding on the bottom when the width is on
- @angelpineda-devSubmitted over 3 years ago
I know that maybe this is simple but you could find something that help me to fix details, so if you can give me a feedback about anything I'd appreciate it.
@trinitysenpaiPosted over 3 years agoNice one ! Some padding on the button and the card as well as spacing bettween the elements(heading, paragraph & button) would be nice.
Marked as helpful0 - @explore-learn-createSubmitted over 3 years ago@trinitysenpaiPosted over 3 years ago
you combined and centered them perfectly :), any reason on using divs rather than the images provided? it would reduce your css code.
0