@juryousSubmitted over 3 years ago
Feedback appreciated
Feedback appreciated
Really nice and clean JS code! Seems to be working well. Did you plan out from the beginning to have the toggle-functionality (when one qa-pair is active, the others are "collapsed")?
I am not satisfied with the broad grid-gap, but I have no idea, how to narrow it.
Hey, looks nice! Like the touch with the shadow changing on mouse-over!
When I inspect your code it looka like "div class="card--supervisor card"" has a margin. Maybe you could take that away first? Same on the opposite card I guess.
If that don't work you could maybe try to (on the div css):
justify-self: end;
Worth a try maybe. https://developer.mozilla.org/en-US/docs/Web/CSS/justify-self