Comment # 4 on bug 1136295 from
In addition to updating the Makefile.am, we will also need to update the icon
search (png|jpg) to 'svg' since yast2-theme is now installing "sclaleable" svg
icons instead of .png. So for instance in yastmodules.cpp:251 there will need
to be an update to

    QRegExp extension( "\\.(png|jpg)$", false );

>From my search, the list of changes will need to be:

grep -n png *
Makefile.am:20:self_icon_DATA=yast.png
Makefile.in:263:self_icon_DATA = yast.png
y2controlcenter.cpp:43:    QString iconName( ICON_DIR "/yast.png" );
y2controlcenterview.cpp:63:    // show titlebar only if title-bar-gradient.png
exists
y2controlcenterview.cpp:64:    if ( !QPixmap( PIXMAP_DIR
"/title-bar-gradient.png" ).isNull() )
y2controlcenterview.cpp:206:    QPixmap titleBarGradientPixmap = QPixmap(
PIXMAP_DIR "/title-bar-gradient.png" );
y2controlcenterview.cpp:220:    QPixmap leftLogo( PIXMAP_DIR
"/title-bar-left.png" );
y2controlcenterview.cpp:246:    QPixmap rightLogo( PIXMAP_DIR
"/title-bar-right.png" );
yastmodules.cpp:55:#define DEFAULT_GROUP_ICON   "yast-default-group.png"
yastmodules.cpp:251:    QRegExp extension( "\\.(png|jpg)$", false );    // case
insensitive
yastmodules.cpp:253:    if ( icon.find( extension ) < 0 )       // no .png or
.jpg extension?
yastmodules.cpp:254:    icon += ".png";                 // assume .png
yastmodules.cpp:326:    QRegExp extension( "\\.(png|jpg)$", false );    // case
insensitive
yastmodules.cpp:328:    if ( icon.find( extension ) < 0 )       // no .png or
.jpg extension?
yastmodules.cpp:329:    icon += ".png";                 // assume .png
yastmodules.cpp:540:    iconfilename = "default.png";

The y2controlcenterview.cpp files are likely not icon related but I am not for
sure. I can build yast2-control-center-qt3 locally, but buildservice fails with
"unresolvable" errors because it can't find qt3-devel at
http://download.opensuse.org/repositories/KDE:/KDE3/openSUSE_Leap_15.1/x86_64/qt3-devel-3.3.8c-lp151.151.2.x86_64.rpm.

I think we are close.


You are receiving this mail because: