Description: Icon "Display Views" missing within Impress' toolbar (see screenshot) while using Sifr icon set. Icons for the different types of views seem to be missing too. Steps to Reproduce: (1) Open Impress Actual Results: Icon "Display Views" as well as icons for view types are missing and replaced by icons from other icon sets. Expected Results: Sifr contains "Display View" icon as well as icons for view types. Reproducible: Always User Profile Reset: No OpenGL enabled: Yes Additional Info: Issue exists using macOS as well as using Linux at least.
Created attachment 151687 [details] Screenshot
Created attachment 151697 [details] Sifr icons for Impres display modes toolbox -- current master/6.3.0alpha1+ Those are Colibre icons (Sifr has a fallback to Colibre, right?). But on Windows build with Sifr icon theme we get the attached. Andreas?
Are there Sifr theme icons used for the Impress displaymodes? BMP_DISPLAYMODE_SLIDE "sd/res/displaymode_slide.png" BMP_DISPLAYMODE_OUTLINE "sd/res/displaymode_outline.png" BMP_DISPLAYMODE_NOTES "sd/res/displaymode_notes.png" BMP_DISPLAYMODE_SLIDE_SORTER "sd/res/displaymode_slidesorter.png" BMP_DISPLAYMODE_SLIDE_MASTER "sd/res/displaymode_slidemaster.png" BMP_DISPLAYMODE_NOTES_MASTER "sd/res/displaymode_notesmaster.png" BMP_DISPLAYMODE_HANDOUT_MASTER "sd/res/displaymode_handoutmaster.png"
Comment on attachment 151697 [details] Sifr icons for Impres display modes toolbox -- current master/6.3.0alpha1+ these Sifr appropriate icons are shown in Windows build of master/6.3.0alpha1+ Version: 6.3.0.0.alpha1+ Build ID: 4b893906984a4620fda7e1914f0d1ea3416ef42c CPU threads: 4; OS: Windows 10.0; UI render: default; VCL: win; Locale: en-US (en_US); UI-Language: en-US Calc: threaded
Icons for display modes have been replaced for all icon sets (second icon on the right of attached screenshot 1), but after placing mouse over icon selection outline is too large now. This is not visible on screenshot, but easy to reproduce by placing mouse over icon (at least on macOS). After changing icon set to verify whether issue occurs with other icon sets too, there are further issues: (1) LibreOffice crashes (see screenshot 2) (2) Recovery dialog appears unselected, Text on button "Start" is white although not selected (see screenshot 3)
Created attachment 152933 [details] Screenshot 1
Created attachment 152934 [details] Screenshot 2
Created attachment 152935 [details] Screenshot 3
Since crash during changing icon style is resolved, behaviour is now better visible: (1) After starting impress from Startcenter wrong icon "Display Views" is displayed as in screenshot 1 independed of icon style. (2) After changing icon style (without crash now) correct icon "Display Views" is displayed, for Sifr as in Stuart's screenshot. (3) Closing Impress and starting Impress again causes wrong icon to be displayed again as in screenshot 1.
@Tor, do you have a few cycles to look at this missing icon fall back on macOS to maybe tweak early in 6.3.0? Thnaks!
Icon seems to be missing in the following icon sets: Sifr, Colibre. Sifr uses Breeze as fallback, Colibre as no sufficient fallback (icon to tall, not from a theme icon set like Breeze, Elementary, Karasa Jaga, Tango).
rizmut committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/fef40fd13ec1739f5308d1ff3bcc2291e7a5d96f Sifr: Fix for tdf#125510 missing highly visible icons It will be available in 7.0.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-4": https://git.libreoffice.org/core/commit/f768f8f3b67af5cee39092e4f6f378bee845b3f4 Sifr: Fix for tdf#125510 missing highly visible icons It will be available in 6.4.2. 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.
Great work - Sifr becomes more and more usable! See attached screenshots 4 to 7 to find some remaining issues: (1) In Writer new Display Views icon is too large (see screenshot 4, red box). Required size is shown in screenshot 5, red box to compare. (2) In Draw there is an out of bounds icon too (see screenshot 6). (3) It would be very nice to get rid of Breeze fallbacks. Icons to illustrate password protected files within Startcenter have no Sifr counterparts currently (see screenshot 7).
Created attachment 158123 [details] Screenshot 4
Created attachment 158124 [details] Screenshot 5
Created attachment 158125 [details] Screenshot 6
Created attachment 158126 [details] Screenshot 7
(In reply to Thorsten Wagner from comment #14) > Great work - Sifr becomes more and more usable! > > See attached screenshots 4 to 7 to find some remaining issues: > > (1) In Writer new Display Views icon is too large (see screenshot 4, red > box). Required size is shown in screenshot 5, red box to compare. I know some icons use the size just like screenshot 5, but I follow Colibre and Sukapura in this case. If you wish, I can change them to larger size as your request just like in Breeze, elementary and Karasa Jaga. Just wait for the patch. > (2) In Draw there is an out of bounds icon too (see screenshot 6). > What do you mean with out of bounds? Please file a separate bug report to keep the patch atomic. > (3) It would be very nice to get rid of Breeze fallbacks. Icons to > illustrate password protected files within Startcenter have no Sifr > counterparts currently (see screenshot 7). I've reported this also in tdf#130898 but we have a blocker: tdf#130936 that prevent me to do the work as if I change them to be monochrome, then the MIME type overlays icons in Start Center become unclear.
> (3) It would be very nice to get rid of Breeze fallbacks. Icons to > illustrate password protected files within Startcenter have no Sifr > counterparts currently (see screenshot 7). The idea is to set all the icons to be monochrome, but if you wish I could help to draw some of them first especially for protected files.
(In reply to Rizal Muttaqin from comment #19) > (In reply to Thorsten Wagner from comment #14) > > Great work - Sifr becomes more and more usable! > > > > See attached screenshots 4 to 7 to find some remaining issues: > > > > (1) In Writer new Display Views icon is too large (see screenshot 4, red > > box). Required size is shown in screenshot 5, red box to compare. > > I know some icons use the size just like screenshot 5, but I follow Colibre > and Sukapura in this case. If you wish, I can change them to larger size as > your request just like in Breeze, elementary and Karasa Jaga. Just wait for > the patch. Indeed icons of Colibre and Sukapura have this issue too. A smaller size as in screenshot 5 is required, because selection rectangle is not drawn completely when moving mouse over. > > (2) In Draw there is an out of bounds icon too (see screenshot 6). > > > > What do you mean with out of bounds? Please file a separate bug report to > keep the patch atomic. "Out of bounds" means icons of the same too large size as discussed above. Size is too large to draw selection rectangle completely when moving mouse over. > > (3) It would be very nice to get rid of Breeze fallbacks. Icons to > > illustrate password protected files within Startcenter have no Sifr > > counterparts currently (see screenshot 7). > > I've reported this also in tdf#130898 but we have a blocker: tdf#130936 that > prevent me to do the work as if I change them to be monochrome, then the > MIME type overlays icons in Start Center become unclear.
(In reply to Rizal Muttaqin from comment #20) > > (3) It would be very nice to get rid of Breeze fallbacks. Icons to > > illustrate password protected files within Startcenter have no Sifr > > counterparts currently (see screenshot 7). > > The idea is to set all the icons to be monochrome, but if you wish I could > help to draw some of them first especially for protected files. Current build from master contains Sifr icons for protected files now - thank you very much!
> > Indeed icons of Colibre and Sukapura have this issue too. A smaller size as > in screenshot 5 is required, because selection rectangle is not drawn > completely when moving mouse over. > "Out of bounds" means icons of the same too large size as discussed above. > Size is too large to draw selection rectangle completely when moving mouse > over. Please file two separate bug reports for these two different requests. We would love to see every patch targets specific report so we can easily avoid conflicted patch when doing back port. This bug report has been renamed to make it clearer what's missing.
Rizal Muttaqin committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/4a670f4284cd2131845ef940d10daedf57014ea4 Sifr: Update Display View icons tdf#125510 It will be available in 7.0.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.
Rizal Muttaqin committed a patch related to this issue. It has been pushed to "libreoffice-6-4": https://git.libreoffice.org/core/commit/57fee3e548cc23163c0d6d5c8be63f6a1336cf8a Sifr: Update Display View icons tdf#125510 It will be available in 6.4.4. 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.
Thank you very match, icon size is correct now for Sifr - will file two similar issues for Colibre and Sukapura
(In reply to Thorsten Wagner from comment #26) > Thank you very match, icon size is correct now for Sifr - will file two > similar issues for Colibre and Sukapura And one another bug report for out of bonds Sifr. I change this bug to RESOLVED FIXED as the bug and the patch are actually exist.
yes, for Sifr there is a remainig icon within Draw toolbar whose size is too large as well