vanilla js, SASS, BEM notation

Community feedback

• 

  MikevPeeren•2,100

  @MikevPeerenPosted almost 3 years ago

  Hey 👋

  Good job, my only comment would be to add a transition to the share modal so it appears nicely in view.

  Marked as helpful

  0
• 

  MichalTrubini•1,220

  @MichalTrubiniPosted almost 3 years ago

  Hi David,

  Instead of switching between "display:none/display:block", I set the opacity to 0 by default. When a user clicks on the button, the opacity is set to 1 with "transition: opacity 0.5s".

  ;-)

  0
• 

  MichalTrubini•1,220

  @MichalTrubiniPosted almost 3 years ago

  Hey Mike,

  Thanks for the suggestion! Animation is now there.

  0

  ---

  Account deleted

  @MichalTrubini Nice animation, how did you manage to do that?

  0