[GitHub] [guacamole-client] jmuehlner commented on a diff in pull request #809: GUACAMOLE-926: Batch Connection Import

[via GitHub]

jmuehlner commented on code in PR #809:
URL: https://github.com/apache/guacamole-client/pull/809#discussion_r1147917617


##########
guacamole/src/main/frontend/src/images/action-icons/guac-monitor-add-many.svg:
##########


Review Comment:
   I do like the look of those better than the current icon, but unfortunately 
they don't render well in actual use on the button. The first one looks pretty 
similar to the current button, with a bit more hair on top.
   <img width="320" alt="image" 
src="https://user-images.githubusercontent.com/4633119/227606540-588cf08e-884a-4ee5-8e26-64708145fe5d.png";>
   
   And the bottom one just looks like a blur to me.
   <img width="323" alt="image" 
src="https://user-images.githubusercontent.com/4633119/227605833-2a20ef6c-48b9-42d3-a48b-d5ded360cd4a.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.

To unsubscribe, e-mail: dev-unsubscr...@guacamole.apache.org

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