Created attachment 145117 [details] comparison LibreOffice 6.0.6 and LibreOffice 6.2 Maybe it's just with gtk3 3.18... Steps to Reproduce: 1. Open Writer 2. Insert a shape 3. Right click on the shape -> Wrap -> Edit -> see attached image Reproduced in Version: 6.2.0.0.alpha0+ Build ID: 66232248ff55639052ddb76918d555e21dc9c46b CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk3; Locale: ca-ES (ca_ES.UTF-8); Calc: threaded
Recent regression. https://cgit.freedesktop.org/libreoffice/core/commit/?id=f6ab12d5dae1212b0ad4e82ca3325f3dd4c974b6 author Caolán McNamara <caolanm@redhat.com> 2018-09-20 14:24:55 +0100 committer Caolán McNamara <caolanm@redhat.com> 2018-09-20 18:13:05 +0200 commit f6ab12d5dae1212b0ad4e82ca3325f3dd4c974b6 (patch) tree 921d7e9de1fe7e323543489f85cd0f78f664057d parent 11a229c0094979b4980d8e3be8a8a7db60d01be7 (diff) weld SwWrapDlg Bisected with: bibisect-linux64-6.2 Adding Cc: to Caolán McNamara
I think this is an icon theme thing, rather than a weld thing. If you change the icon theme to galaxy as used in the 6-0 screenshot you get big icons. In the elementary theme if you select "before" and toggle contour some icons have a different size to others. Checking I get... file icon-themes/elementary/sw/res/wr*.png icon-themes/elementary/sw/res/wr010.png: PNG image data, 16 x 16, 4-bit colormap, non-interlaced icon-themes/elementary/sw/res/wr011.png: PNG image data, 16 x 16, 4-bit colormap, non-interlaced icon-themes/elementary/sw/res/wr01.png: PNG image data, 24 x 24, 8-bit colormap, non-interlaced icon-themes/elementary/sw/res/wr02.png: PNG image data, 16 x 16, 4-bit colormap, non-interlaced icon-themes/elementary/sw/res/wr03.png: PNG image data, 24 x 24, 8-bit colormap, non-interlaced icon-themes/elementary/sw/res/wr04.png: PNG image data, 16 x 16, 4-bit colormap, non-interlaced icon-themes/elementary/sw/res/wr05.png: PNG image data, 16 x 16, 4-bit colormap, non-interlaced icon-themes/elementary/sw/res/wr06.png: PNG image data, 24 x 24, 8-bit colormap, non-interlaced icon-themes/elementary/sw/res/wr07.png: PNG image data, 16 x 16, 4-bit colormap, non-interlaced icon-themes/elementary/sw/res/wr08.png: PNG image data, 16 x 16, 4-bit colormap, non-interlaced icon-themes/elementary/sw/res/wr09.png: PNG image data, 24 x 24, 8-bit colormap, non-interlaced some are 24x24 and some are 16x16, they should ideally all be the same size, the larger size. (In the colibre theme they are 32x32, and the breeze_dark theme has these mixed sizes too)
caolanm->andreas_kainz: Do you think we can get all of those wr*png icons in the larger size ? There's a few other problems there too on toggling contour, "After" looks good in both modes, large rectangle vs large diamond, but we don't get diamonds in the other contour cases.
Right now I'm trying to help Andreas for maintaining elementary icon and I have send a patch for this bug. I've changed color of "object" to purple for all size (including the new 32px one). Just waiting for approval. https://gerrit.libreoffice.org/#/c/60950/
Rizal Muttaqin committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=7f16835c10435ddafdfe05921eac9b89e488ab8e tdf#120065 and tdf#120108, Fix wrap elementary icons It will be available in 6.2.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.
Looks an improvement to me. Though in the wrap page of format->image, selecting e.g. optimal, and toggling contour on/off shows the same icon for optimal with contour on or off. I wonder if there should be a contour version of that icon, e.g. sw/res/wr06.png vs sw/res/wr011.png. There is also no difference between sw/res/wr01.png (wrap NONE) and sw/res/wr07.png (coutour wrap NONE)
(In reply to Caolán McNamara from comment #6) > Looks an improvement to me. Though in the wrap page of format->image, > selecting e.g. optimal, and toggling contour on/off shows the same icon for > optimal with contour on or off. I wonder if there should be a contour > version of that icon, e.g. sw/res/wr06.png vs sw/res/wr011.png. There is > also no difference between sw/res/wr01.png (wrap NONE) and sw/res/wr07.png > (coutour wrap NONE) @Rizal Muttaqin, Would you mind taking a look to Caolán's Comment ?
Removing the UX keyword
Dear Rizal Muttaqin, This bug has been in ASSIGNED status for more than 3 months without any activity. Resetting it to NEW. Please assigned it back to yourself if you're still working on this.
(In reply to Caolán McNamara from comment #6) > Looks an improvement to me. Though in the wrap page of format->image, > selecting e.g. optimal, and toggling contour on/off shows the same icon for > optimal with contour on or off. I wonder if there should be a contour > version of that icon, e.g. sw/res/wr06.png vs sw/res/wr011.png. There is > also no difference between sw/res/wr01.png (wrap NONE) and sw/res/wr07.png > (coutour wrap NONE) Yes, Galaxy use diamond to represent contour on/off. will try to set different icons as Galaxy has. As a side note TBH, I personally a little confuse with wrapping in LibO, For example wrap through the icon it self show image under text but in reality it vice versa. Could someone please see wrapping icons in Karasa Jaga theme to see if the object representation (as leaf) is not suitable?
rizmut committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/4ebedd4f21c6a54018e7b7692b89fa46263de756%5E%21 elementary: tdf#120065, differentiate contour on/off wrap icons It will be available in 6.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
rizmut committed a patch related to this issue. It has been pushed to "libreoffice-6-3": https://git.libreoffice.org/core/+/e34e63bada32b18b2fac373508457114f478c0c7%5E%21 elementary: tdf#120065, differentiate contour on/off wrap icons It will be available in 6.3.0.1. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.