Bug 193770

Summary:	toolbar icon for magnifier is wrong
Product:	[Applications] okular	Reporter:	M G Berberich <kde>
Component:	general	Assignee:	Okular developers <okular-devel>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	0.8.2
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	general zoom icon

Description M G Berberich 2009-05-23 14:41:39 UTC

Version:           0.8.2 (using 4.2.2 (KDE 4.2.2), Debian packages)
Compiler:          cc
OS:                Linux (x86_64) release 2.6.29.1-hermione

the toolbar-icon for magnifying glass is “zoom-original”, which is irritating.

Comment 1 Pino Toscano 2009-05-23 14:49:04 UTC

Suggestions welcome.

Comment 2 M G Berberich 2009-05-23 15:57:09 UTC

Created attachment 33949 [details]
general zoom icon

I'm not an artist, but removing the “1” from the icon is no problem and gives a general zoom-icon. See attachment.

idealy, this should be added to the oxygen set of course.

Comment 3 M G Berberich 2009-05-23 15:58:49 UTC

Silly me.
I just saw it's already there with the name “page-zoom”

Comment 4 Pino Toscano 2009-05-24 17:58:59 UTC

(In reply to comment #3)
> I just saw it's already there with the name “page-zoom”

Ah ok, it wasn't there when the code related to that icon was ported to KDE 4, and didn't check since that.
Fix coming in a minute, thanks.

Comment 5 Pino Toscano 2009-05-24 18:09:25 UTC

SVN commit 972330 by pino:

use the proper "page-zoom" icon for the zoom tool and the zoom selector
BUG: 193770


 M  +2 -2      pageview.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=972330

Comment 6 Pino Toscano 2009-05-24 18:10:52 UTC

SVN commit 972331 by pino:

backport: use the proper "page-zoom" icon for the zoom tool and the zoom selector
CCBUG: 193770


 M  +2 -2      pageview.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=972331