Bug 429189 - hardcoded pixmaps
Summary: hardcoded pixmaps
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: pirut
Version: 8
Hardware: All
OS: Linux
urgent
low
Target Milestone: ---
Assignee: Jeremy Katz
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: F9Blocker
TreeView+ depends on / blocked
 
Reported: 2008-01-17 20:17 UTC by Jakub 'Livio' Rusinek
Modified: 2008-01-31 16:30 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-01-31 16:30:51 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
screeny of pirut (234.15 KB, image/png)
2008-01-17 20:17 UTC, Jakub 'Livio' Rusinek
no flags Details
screeny of system-install-packages (234.99 KB, image/png)
2008-01-17 20:18 UTC, Jakub 'Livio' Rusinek
no flags Details

Description Jakub 'Livio' Rusinek 2008-01-17 20:17:10 UTC
Description of problem:
pirut and system-install-packages don't use icons provided by desktop environment.

pirut:
browse icon should be gtk-index @ 32x32
search => gtk-find or edit-find @ 32x32
list => gtk-sort-ascending @ 32x32

system-install-packages:
main icon => gtk-harddisk or better package-x-generic @ 64x64 (I guess)




Remember, apps should integrate as good as possible with environment, where
they're supposed to run.

Comment 1 Jakub 'Livio' Rusinek 2008-01-17 20:17:10 UTC
Created attachment 292060 [details]
screeny of pirut

Comment 2 Jakub 'Livio' Rusinek 2008-01-17 20:18:01 UTC
Created attachment 292062 [details]
screeny of system-install-packages

Comment 3 Jeremy Katz 2008-01-17 22:40:03 UTC
I made a first pass through these a couple of months ago... I'll look at these
in a couple of weeks (my time is a bit constrained at the moment and so I'm not
having much time for Fedora work)

Comment 4 Jakub 'Livio' Rusinek 2008-01-17 22:46:37 UTC
If you'll do something, post here screenshot of patched pirut (; .

Comment 5 Jeremy Katz 2008-01-31 16:30:51 UTC
Fixed up in git, will be in 1.2.30


Note You need to log in before you can comment on or make changes to this bug.