All comments

• 

  Jurijus•300

  @juryousSubmitted almost 4 years ago

  Feedback appreciated

  

  Mobile first, vanilla css/js

  2

  41
  

  Sebastian T•65

  @Seti108Posted almost 4 years ago

  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")?

  0

  This comment has received 1 replyShow
• 

  Sven•295

  @Sven72Submitted about 4 years ago

  I am not satisfied with the broad grid-gap, but I have no idea, how to narrow it.

  

  Using Flex and Grid. Grid-Gap is not optimal.

  1

  20
  

  Sebastian T•65

  @Seti108Posted about 4 years ago

  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

  1