stevecrox commented on PR #39: URL: https://github.com/apache/maven-fluido-skin/pull/39#issuecomment-1166242124
So .. knowing its only 116 x 116 on a 4k screen gave me a target. One of the factors I've included is accessibility I have worked with several people with sight issues and a common seems to be a 48" monitor combined with a few different ways of adding zoom on top. While working on the bootstrap upgrades I've been checking layout by reducing it to <578px width (for phones) and also leaving a normal layout but zoomed to 400% (it results in a display size similar to my co-workers setup). - Using Gwenview reducing it to 5% (122 x 122) cut the size down to 14Kib, then zooming into 400% it looked like a blocky mess. - Using Gwenview reducing it to 10% (245 x 245) cut the size down to 26Kib, then zooming into 400% it starts getting blocky but looks ok. - Using Gwenview reducing it to 15% (367 x 367) cut the size down to 44Kib, then zooming into 400% it looks pretty good but is larger than my screen I did a quick build of the Integration Tests and in v2.8 the side bar takes the entire screen width so checking with Gwenview was pretty accurate. Is there any chance you could get the image size at a 400% zoom? I'm inclined to go with the 10% sized image (which I've committed). -- 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: issues-unsubscr...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org