Bug 119867 - different icon types used by slideshow for for backwards/forwards.
Summary: different icon types used by slideshow for for backwards/forwards.
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Plugin-Generic-Presentation (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-01-10 18:05 UTC by S. Burmeister
Modified: 2018-03-23 21:04 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In: 0.1.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description S. Burmeister 2006-01-10 18:05:21 UTC
Version:           0.80 (using KDE KDE 3.5.0)
Installed from:    SuSE RPMs
OS:                Linux

When I run the slideshow, I notice that I have two different kinds of arrows for back and forward.

Back uses the icon-type used in konqueror's toolbar, forward the one used in kmail's toolbar to navigate the messages.

To make it look nicer digikam should just use the icons used in konqueror's toolbar.
Comment 1 Thiago Macieira 2006-01-10 18:20:58 UTC
Digikam is using the icons "back" and "forward". That's the same Konqueror uses.

KMail uses "previous" and "next".

Everything in Subversion revision 487700.
Comment 2 Joern Ahrens 2006-01-10 18:34:48 UTC
SVN commit 496515 by jahrens:


He is talking about the slideshow, where we used indeed
different icons.

BUG: 119867


 M  +1 -1      toolbar.cpp  


--- trunk/extragear/libs/kipi-plugins/slideshow/toolbar.cpp #496514:496515
@@ -45,7 +45,7 @@
     KIconLoader* loader = kapp->iconLoader();
     m_playBtn->setIconSet(loader->loadIcon("player_pause", KIcon::NoGroup, 22));
     m_prevBtn->setIconSet(loader->loadIcon("back", KIcon::NoGroup, 22));
-    m_nextBtn->setIconSet(loader->loadIcon("next", KIcon::NoGroup, 22));
+    m_nextBtn->setIconSet(loader->loadIcon("forward", KIcon::NoGroup, 22));
     m_stopBtn->setIconSet(loader->loadIcon("stop", KIcon::NoGroup, 22));
 
     lay->addWidget(m_playBtn);
Comment 3 S. Burmeister 2006-07-20 20:09:26 UTC
This still seems to be valid for 0.82 slideshow. I use the nuvola icon-theme which should not really make a difference.

I am talking about the buttons Pause/Close/Next/Previous that pop-up if you move the mouse to some corner of the screen while the slideshow is on.