bug fix: #589 (added visibility tansition to .top-button to really hide it)

[JWestarp]

Closes #589

[crftwrk]

Thank you for pointing this out!

Just 2 tiny things:

1 - We can skip visibility: visible; because Bootstrap has

.visible {
  visibility: visible !important;
}

in utilities.

2 - Using .3s is cleaner than 300ms and used in the entire theme.

Overall code:

.top-button {
  right: 5%;
  bottom: 30px;
  opacity: 0;
  visibility: hidden;
  transition: visibility 0s linear .3s, opacity .3s;

  &.visible {
    opacity: 1;
    transition: visibility 0s linear 0s, opacity .3s;
  }
}

If you want to change this quickly, we are ready to merge.

[JWestarp]

@crftwrk Thanks for the quick reply. Yes, i agree with you.
I just changed that.

[crftwrk]

👍