[GitHub] [couchdb-fauxton] Antonio-Maranhao commented on pull request #1265: Replace notification bar by notification toasts

[GitBox <gi...@apache.org>]

Antonio-Maranhao commented on pull request #1265:
URL: https://github.com/apache/couchdb-fauxton/pull/1265#issuecomment-619689297


   @popojargo you're definitely on the right track here - I love the ESC to dismiss it !!!
   I find the colored background doesn't really match the rest of Fauxton's design. I'd prefer something closer to the original notification bar:
   
   ![image](https://user-images.githubusercontent.com/30349380/80330207-90e5f580-8812-11ea-9ab2-8d1ab56cefce.png)
   
   which you can get with: 
   ```
   .Toastify__toast--info {
     &, .Toastify__close-button--info {
       border: 1px solid #999;
       border-left: 6px solid @infoAlertBackground;
       border-right: 0;
       background-color: #f1f1f1;
       color: black;
     }
   }
   ```
   
   Also, my preference is for `opacity: 1` always (i.e. no transparency), otherwise it's harder read the notification text when the background already has some text/elements. E.g.:
   
   ![image](https://user-images.githubusercontent.com/30349380/80330855-61d08380-8814-11ea-8a0d-9d1ae08707ca.png)
   
   
   ![image](https://user-images.githubusercontent.com/30349380/80330864-69902800-8814-11ea-85d5-033760064c3a.png)
   
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org