Review Request 127201: Fix svg icon path resolving in IconItem

Xuetian Weng wengxt at
Mon Feb 29 21:05:54 UTC 2016

This is an automatically generated e-mail. To reply, visit:

(Updated Feb. 29, 2016, 11:05 p.m.)


This change has been marked as submitted.

Review request for KDE Frameworks, Plasma, Kai Uwe Broulik, David Rosca, and Marco Martin.


Submitted with commit 066cf0a0d7b46c60fccb945dbc0e9f858c4d6967 by Weng Xuetian to branch master.

Repository: plasma-framework


Well.. the bug should be obvious:
1. QString iconPath overrides the iconPath in a higher scope. introduced in
2. iconPath not assigned, also introduced in
3. svgz -> svg?? introduced in


  autotests/data/bug359388/hicolor/22x22/apps/bug359388.svg PRE-CREATION 
  autotests/data/icons/hicolor/22x22/apps/bug359388.svg 8739af5 
  autotests/data/icons/test-theme/apps/22/tst-plasma-framework-test-icon.svg PRE-CREATION 
  autotests/data/icons/test-theme/apps/32/tst-plasma-framework-test-icon.svg PRE-CREATION 
  autotests/data/icons/test-theme/apps/48/konversation.svg PRE-CREATION 
  autotests/data/icons/test-theme/index.theme PRE-CREATION 
  autotests/iconitemtest.h 9fd3063 
  autotests/iconitemtest.cpp ea98c9d 
  src/declarativeimports/core/iconitem.cpp c65a9a7 



tested with qmlscene with Plasma.IconItem.


Xuetian Weng

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the Plasma-devel mailing list