[Bug 199681] Review Request: slab

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: slab


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=199681





------- Additional Comments From sundaram at redhat.com  2006-07-31 20:47 EST -------
(In reply to comment #28)

> This i can gladly do, though looking at the patches they might be called
> 'intrusive'. The meat of those patches is a new lib (for tracking recently used)
> but there is also a part which rewrites some of the application launching code,
> which ofcource is anything but trivial .. if it would ever break, there would be
> no desktop worth mentioning for the end users :-)
> 
> For the time being i've replaced the recently used with places, but i'll file
> the bugs with the RFE's, and see how it floats.. if there accepted i'll build it
> so that it has both recently used and places (seeing how nautilus has this as a
> prominent thing too, we might as well include it)

Right. Places seems the best option if you dont manage to get the other patches
in which seems unlikely from your comments. I would actually prefer places as a
option even if we have a recently used apps feature. 


> Yelp and beagle are definatly 2 packages that will make the requires list,
> however I'm still weighing if i should expand the requires to include openoffice
> writer/impress/calc, firefox and evolution ... it comes hardcoded with a list of
> pre-set 'favorite applications' and it would be a shame to have this menu be
> empty (ie .desktop files not found for those apps).. however it might be a (to)
> heavy dependency to pull in?

Apparently the favorite applications are treated as bookmarks and assuming the
applications in the default desktop group to be present in the system is very
reasonable. I would prefer these apps to be not defined as dependencies. Nothing
would happen if the user clicks on a non installed application currently which
is less of a evil (maybe a quick dialog box that its not installed or offering
to install it using pirut might be a option) compared to pulling in the the
kitchen sink as dependencies. 


> 

> I'll have to ping Jeremy for this, no docs make any mention of such
> functionality, and where there is a system-install-packages, there's no
> uninstall .. Alternativly for the time being i considered patching out those
> menu options if we can't provide them with functionality for now

Ok. That makes sense.


> 
> > I am just a volunteer from the Fedora perspective so take your time. Just
> > reviewing the functionality. 
> Sorry if i sounded a bit grumpy before, just been a long day :-)
> 

Thats fine. People assume everyone with a @redhat id has a Fedora job. Just
clarifying. 

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.