Description: On high resolution (3200x1800) icon for "read only" files is too small. Attached two pictures, same machine on 1920x1080 resolution, and high 3200x1800 resolution. Steps to Reproduce: Open read only file on high resolution computer. Actual Results: Icon too small. Expected Results: Regular size icon. Reproducible: Always User Profile Reset: No Additional Info: User-Agent: Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:55.0) Gecko/20100101 Firefox/55.0
Created attachment 135573 [details] 1920x1080 resolution
Created attachment 135574 [details] 3200x1080 resolution
Created attachment 135589 [details] clip from Writer session full screen on 4K monitor Can not confirm on Windows 10 Pro 64-bit en-US build Version: 5.4.0.3 (x64) Build ID: 7556cbc6811c9d992f4064ab9287069087d7f62c CPU threads: 8; OS: Windows 6.19; UI render: GL; Locale: en-US (en_US); Calc: group The infobox.svg icon is rendered well with either OpenGL or default rendering
Could not confirm on Windows builds, but also could not find the .svg flavor of the icons just the .png =-ref-= https://opengrok.libreoffice.org/xref/core/sfx2/source/dialog/infobar.cxx#73
Hello, My examples are taken on Windows 8.1 machine. It's 13,3" laptop with 3200x1800 default resolution. But I see windows reports different resolution to web pages then default, maybe that's root problem here. If you need more testing, just write what you need.
heiko tietze committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=4438849e400ecef3cdf2f267dc75a76d4b11aba1 tdf#111844 infobar icons on high dpi It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Mikeyy, please retest the issue on your highdpi screen. The nightly build available tomorrow should include the patch. For 64bit Windows use http://dev-builds.libreoffice.org/daily/master/Win-x86_64@42/current/ - please check the date to be Aug/22th.
Created attachment 135722 [details] Re-test 22. 8. 2017. with LO 6.0 alpha (In reply to Heiko Tietze from comment #7) > Mikeyy, please retest the issue on your highdpi screen. The nightly build > available tomorrow should include the patch. > As you can see in attached test picture, icon looks better now, but text is much closer to icon then it is with LO 5.4 on regular resolution.
(In reply to Mikeyy - L10n HR from comment #8) > As you can see in attached test picture, icon looks better now, but text is > much closer to icon then it is with LO 5.4 on regular resolution. Grrr, forgot the text position ;-). Gerrit is compiling...
Heiko Tietze committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=4d60d96a22ef2f67db7c7e99981447bd81f776c4 tdf#111844 infobar icons/text on high dpi It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Thumbs up! Looks good. If it's not too much trouble, you can port it back to 5.4.
heiko tietze committed a patch related to this issue. It has been pushed to "libreoffice-5-4": http://cgit.freedesktop.org/libreoffice/core/commit/?id=9d7b4b0114fdbd784ab87ab532140af9e1bbc06d&h=libreoffice-5-4 tdf#111844 infobar icons on high dpi It will be available in 5.4.2. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
I’ve squashed both master commits into one and pushed that into libreoffice-5-4.