Bug 193770 - toolbar icon for magnifier is wrong
Summary: toolbar icon for magnifier is wrong
Status: RESOLVED FIXED
Alias: None
Product: okular
Classification: Applications
Component: general (show other bugs)
Version: 0.8.2
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Okular developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-05-23 14:41 UTC by M G Berberich
Modified: 2009-05-24 18:10 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:

Attachments
general zoom icon (19.17 KB, image/svg+xml-compressed)
2009-05-23 15:57 UTC, M G Berberich 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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