| Summary: | composition and aggregation toolbar icons are incorrect | ||
|---|---|---|---|
| Product: | [Applications] umbrello | Reporter: | Maxim Egorushkin <maxim.yegorushkin> |
| Component: | general | Assignee: | Lays Rodrigues <laysrodriguessilva> |
| Status: | RESOLVED NOT A BUG | ||
| Severity: | minor | CC: | laysrodriguessilva, ralf.habacker |
| Priority: | NOR | ||
| Version: | 2.16.0 (KDE Applications 15.04.0) | ||
| Target Milestone: | --- | ||
| Platform: | Fedora RPMs | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
| Attachments: |
screenshot of toolbar
umbrello 2.16.3 screenshot Inversion of pixmaps |
||
|
Description
Maxim Egorushkin
2016-04-07 11:49:42 UTC
Yeah, thats true. Ralf, I have searched in the code where this inversion appears, but I wasn't able to find it. The logic of the worktoolbar.cpp and the png files seems right. The pics files are correct. Created attachment 98655 [details]
screenshot of toolbar
In umbrello branch Applications/16.04 it looks okay (see screenshot). May be this bug is a duplicate of a another bug, which solved the issue already. You may inspect git log to find it.
Ok, I will find that and apply to branch Frameworks. Created attachment 98680 [details]
umbrello 2.16.3 screenshot
Cannot reproduce this problem with umbrello 2.16.3. Looks like a positive false
(In reply to Ralf Habacker from comment #5) > Cannot reproduce this problem with umbrello 2.16.3. Looks like a positive false Same with 2.16.0. If there are icons swapped it could only be a packaging or installation issue on the related distribution. If the problem persists, feel free to reopen this bug with informations, which makes it possible to reproduce the problem. Created attachment 98689 [details]
Inversion of pixmaps
This bugs appears for me in Umbrello Version 2.19.0 downloaded from repository and the build from branch frameworks.
Comment on attachment 98689 [details]
Inversion of pixmaps
You need to find out where this swapping come from. This may include basic KF5 libraries.
I had a look at the code, and the method that loads the pixmaps, but I wasn't able to find it. The logic and the name of the files(folder pics) looks right. (In reply to Lays Rodrigues from comment #9) > I had a look at the code, and the method that loads the pixmaps, but I > wasn't able to find it. The logic and the name of the files(folder pics) > looks right. You are refering to umbrello code or to whole related KF5 code e.g. libKF5IconThemes5, which provides icon support for KF5 ? On opensuse distributions for example you have to install several debug packages e.g. by running sudo zypper install kiconthemes-debugsource kiconthemes-devel-debuginfo libKF5IconThemes5-debuginfo on other distributions the package may be named different. After that it is possible to debug into KF5 code used by umbrello. I mean here the third comment that I made. Related to the method that load the pixmaps. So it can be a lib problem? (In reply to Lays Rodrigues from comment #11) > I mean here the third comment that I made. Related to the method that load > the pixmaps. > So it can be a lib problem? On a opensuse 13.1system (KDE4 based with kf5 libraries) I do not have the problem with kf5 frameworks build, on another system with opensuse Leap 42.1 (KF5 based) I have, so I would say it is somehow related with KF5 libraries, but cannot say why and where. (In reply to Ralf Habacker from comment #12) > on another system with opensuse Leap 42.1 (KF5 based) I have, > so I would say it is somehow related with KF5 libraries, but cannot say why and where. > running umbrello from framework-branches returns Illegal icon group: 7 Illegal icon group: 7 Illegal icon group: 7 Illegal icon group: 7 Illegal icon group: 7 Overwriting existing old cached entry due to collision. The last message comes from icon cache, in source kcoreaddons/src/lib/caching/kshareddatacache.cpp:1478 (In reply to Ralf Habacker from comment #13) > Illegal icon group: 7 I remember sawing this warnings in older kdelibs version which may indicate an icon cache issue also for 2.16.0 (In reply to Ralf Habacker from comment #12) > on another system with opensuse Leap 42.1 (KF5 based) I have, so I would say it is somehow related with KF5 libraries, but cannot say why and where. umbrello5 on opensuse Leap 42.1 uses breeze icons https://build.opensuse.org/package/show/KDE:Frameworks5/breeze5-icons, which contains swapped svg icong. Git source is from https://quickgit.kde.org/?p=breeze-icons.git&a=tree&h=b628eb33870953f30f18b3b464cd1f3be4c793af&hb=9ec402757e62bae12102d8a8851e9e5989e3645a&f=icons%2Factions%2F22 (In reply to Ralf Habacker from comment #15) > (In reply to Ralf Habacker from comment #12) > > on another system with opensuse Leap 42.1 (KF5 based) I have, so I would say it is somehow related with KF5 libraries, but cannot say why and where. > > umbrello5 on opensuse Leap 42.1 uses breeze icons > https://build.opensuse.org/package/show/KDE:Frameworks5/breeze5-icons, which > contains swapped svg icon. Git source is from > https://quickgit.kde.org/?p=breeze-icons. > git&a=tree&h=b628eb33870953f30f18b3b464cd1f3be4c793af&hb=9ec402757e62bae12102 > d8a8851e9e5989e3645a&f=icons%2Factions%2F22 The kf5 related variant of the icon swapping bug is not a umbrello bug handled by this bug tracker nor could be fixed at umbrello git repo. A bug report and/or patch need to be filed at https://phabricator.kde.org/project/profile/33/. (In reply to Ralf Habacker from comment #16) > A bug report and/or patch need to be filed at https://phabricator.kde.org/project/profile/33/. New bug filed at https://phabricator.kde.org/T2404 (In reply to Ralf Habacker from comment #17) > New bug filed at https://phabricator.kde.org/T2404 @lays: The next step is to checkout the breeze-icons git repo from https://phabricator.kde.org/diffusion/BREEZE/, to edit the related icons and to create a patch at https://phabricator.kde.org/differential/diff/create/ |