Summary: | [thumbnailer] Images files thumbnails generated by calligra have an obstrusive overlay icons | ||
---|---|---|---|
Product: | [Applications] krita | Reporter: | David REVOY <info> |
Component: | General | Assignee: | Krita Bugs <krita-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | minor | CC: | cfeck, halla, kossebau, sven.langkamp |
Priority: | NOR | ||
Version: | git master (please specify the git hash!) | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/calligra/ca51fcfc7b51480da36c0a7562a28da1163fb7e7 | Version Fixed In: | 2.6 |
Sentry Crash Report: |
Description
David REVOY
2012-01-07 13:50:40 UTC
This icon isn't added by Krita or the Calligra thumbnail tool, so it looks like it's done by the filemanager. Maybe it mixed some thumbnails from Dolphin and Nautilus. Thank you Sven for the answer. So I think it's Dolphin. Do you think it's possible to move this thread to the Dolphin project ? If not, please close the bug report ; I 'll open a new one with their correct header. Looking at the calligra thumbnailer, it explicitely requests those icons. https://projects.kde.org/projects/calligra/repository/revisions/master/entry/tools/thumbnail/calligracreator.cpp#L119 return (Flags)(DrawFrame | BlendIcon); I see. So it's a Calligra problem after all. Probably needs separate thumbnailers. Okay, we can confirm then. It shouldn't be too hard to fix, right? Git commit 8b5d90f3db360a8f15e748c2c8cc982fdbc0cdb1 by Boudewijn Rempt. Committed on 05/03/2012 at 19:15. Pushed by rempt into branch 'master'. Don't use the blendicon for krita or openraster files M +6 -1 tools/thumbnail/calligracreator.cpp http://commits.kde.org/calligra/8b5d90f3db360a8f15e748c2c8cc982fdbc0cdb1 I was browsing my old bug reports and I saw this one was in RESO. Here I always had the 'Blendicon' on the thumbnail in file explorer of image file created by Krita. ex : http://img861.imageshack.us/img861/7954/20120107screenshot01.jpg so I reopen this one. Can confirm that the bug is still present. The fix http://quickgit.kde.org/?p=calligra.git&a=commit&h=8b5d90f3db360a8f15e748c2c8cc982fdbc0cdb1 sadly does not work, as the flags() property of the thumbnail plugin is not queried based on a file path, instead assumed to be constant. So with the fix above the flag now depends on the last file for which a preview was made. As Sven already guessed the solution will be two separate thumbnailers. Shall I work on that? Could integrate with the oter fixes on the thumbnailer I am doing. That would be absolutely wonderful! Review request which contains the fix can be found at https://git.reviewboard.kde.org/r/105373/ and waits for review :) It creates two different thumbnail plugins, one with BlendIcon flag and one without. It also installs different thumbnail descriptions (desktop files), so that for .kra and .ora files the non-BlendIcon thumbnailer is used. I also changed the files for all the vector image formats (ODG, xfig) to use the non-BlendIcon thumbnailer, for consistency. After all the SVG thumbnailer (outside of Calligra) also does not set BlendIcon. Git commit ca51fcfc7b51480da36c0a7562a28da1163fb7e7 by Friedrich W. H. Kossebau. Committed on 07/07/2012 at 17:20. Pushed by kossebau into branch 'master'. pimp the thumbnailer * duplicate into two plugins: calligraimagethumbnail for preview without blending the icon (for all image formats), calligrathumbnail for preview with blending the icon (for all other) * wait for result of thumbnail rendering if needed * install separate thumbnail descriptions per import filter and only if the import filter is installed * put a white background behind extracted mimetypes REVIEW: 105373 FIXED-IN: 2.6 M +1 -0 filters/flow/visio/import/CMakeLists.txt A +9 -0 filters/flow/visio/import/flow_vsdx_thumbnail.desktop M +2 -0 filters/karbon/karbon1.x/CMakeLists.txt A +9 -0 filters/karbon/karbon1.x/karbon_karbon1x_thumbnail.desktop M +1 -1 filters/karbon/wmf/CMakeLists.txt A +9 -0 filters/karbon/wmf/karbon_wmf_thumbnail.desktop M +1 -0 filters/karbon/wpg/CMakeLists.txt A +9 -0 filters/karbon/wpg/karbon_wpg_thumbnail.desktop M +1 -1 filters/karbon/xfig/CMakeLists.txt A +9 -0 filters/karbon/xfig/karbon_xfig_thumbnail.desktop M +2 -0 filters/sheets/excel/import/CMakeLists.txt A +9 -0 filters/sheets/excel/import/sheets_excel_thumbnail.desktop M +2 -0 filters/sheets/xlsx/CMakeLists.txt A +9 -0 filters/sheets/xlsx/sheets_xlsx_thumbnail.desktop M +1 -0 filters/stage/kpr2odf/CMakeLists.txt A +9 -0 filters/stage/kpr2odf/stage_kpr_thumbnail.desktop M +1 -0 filters/stage/powerpoint/CMakeLists.txt A +9 -0 filters/stage/powerpoint/stage_powerpoint_thumbnail.desktop M +2 -1 filters/stage/pptx/CMakeLists.txt A +9 -0 filters/stage/pptx/stage_pptx_thumbnail.desktop M +2 -1 filters/words/docx/CMakeLists.txt A +9 -0 filters/words/docx/words_docx_thumbnail.desktop M +1 -1 filters/words/msword-odf/CMakeLists.txt A +9 -0 filters/words/msword-odf/words_msword_thumbnail.desktop M +1 -0 filters/words/rtf/import/CMakeLists.txt A +9 -0 filters/words/rtf/import/words_rtf_thumbnail.desktop M +1 -0 filters/words/wordperfect/import/CMakeLists.txt A +9 -0 filters/words/wordperfect/import/words_wpd_thumbnail.desktop M +1 -0 filters/words/works/import/CMakeLists.txt A +9 -0 filters/words/works/import/words_wps_thumbnail.desktop M +1 -1 krita/kritapart.desktop M +1 -1 krita/plugins/formats/ora/CMakeLists.txt A +9 -0 krita/plugins/formats/ora/krita_ora_thumbnail.desktop M +36 -6 tools/thumbnail/CMakeLists.txt A +9 -0 tools/thumbnail/calligra_odg_thumbnail.desktop M +68 -63 tools/thumbnail/calligracreator.cpp M +11 -10 tools/thumbnail/calligracreator.h D +0 -33 tools/thumbnail/calligrathumbnail.desktop A +9 -0 tools/thumbnail/krita_kra_thumbnail.desktop D +0 -65 tools/thumbnail/otherofficethumbnail.desktop A +9 -0 tools/thumbnail/sheets_ods_thumbnail.desktop A +9 -0 tools/thumbnail/stage_odp_thumbnail.desktop A +9 -0 tools/thumbnail/words_odt_thumbnail.desktop http://commits.kde.org/calligra/ca51fcfc7b51480da36c0a7562a28da1163fb7e7 |