After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 418015 - downloads section does not work with epiphany
downloads section does not work with epiphany
Status: RESOLVED FIXED
Product: gimmie
Classification: Deprecated
Component: general
0.2.4
Other All
: Normal minor
: ---
Assigned To: Tony Tsui
Gimmie Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-03-13 21:40 UTC by David Prieto
Modified: 2007-05-01 00:15 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Support monitoring of Epiphany's Download Directory (2.63 KB, patch)
2007-03-17 06:54 UTC, Tony Tsui
none Details | Review
Simplify code with GConfBridge (2.29 KB, patch)
2007-03-17 10:37 UTC, Tony Tsui
none Details | Review
Handle if Epiphany is not installed (2.67 KB, patch)
2007-03-19 12:08 UTC, Tony Tsui
none Details | Review
Fix crash because of incorrect return value from get_enabled() (2.16 KB, patch)
2007-04-17 12:43 UTC, Tony Tsui
none Details | Review

Description David Prieto 2007-03-13 21:40:44 UTC
Hi,

It's cool to have a downloads section in the library menu, but does it have to work only on Firefox? I'm an Epiphany user, and I would like to be able to make use of that download section or get rid of it altogether.
Comment 1 Alex Graveley 2007-03-13 23:33:19 UTC
Where does epiphany store it's download info?  It doesn't use Mozilla's standard downloads.rdf mechanism?
Comment 2 Christian Persch 2007-03-15 23:43:22 UTC
Not sure what info you want, but Epiphany doesn't store downloads after they are finished.
Comment 3 Tony Tsui 2007-03-17 06:54:48 UTC
Created attachment 84754 [details] [review]
Support monitoring of Epiphany's Download Directory

We can get Epiphany's download location from the gconf key /apps/epiphany/directories/downloads_folder

Attached is a patch which adds support to DownloadsSource for monitoring Epiphany's download directory.

I'm not sure if this is the best way to implement monitoring. Feedback welcomed.
Comment 4 Alex Graveley 2007-03-17 07:43:41 UTC
This looks pretty good tony.  I think using a GConfBridge with "/apps/epiphany/directories/" set as the prefix would clean up this code a lot though.
Comment 5 Tony Tsui 2007-03-17 10:37:00 UTC
Created attachment 84760 [details] [review]
Simplify code with GConfBridge

Simplified patch with GConfBridge.

I didn't realise GConfBridge could be used this way. Shall I do the same to how ItemIconView gets Nautilus' click policy preference from /apps/nautilus/preferences/?
Comment 6 Tony Tsui 2007-03-19 12:08:41 UTC
Created attachment 84874 [details] [review]
Handle if Epiphany is not installed

I've updated the patch to handle the case where the gconf key is not available because Epiphany is not installed.
Comment 7 Tony Tsui 2007-04-17 12:43:38 UTC
Created attachment 86495 [details] [review]
Fix crash because of incorrect return value from get_enabled()
Comment 8 Alex Graveley 2007-05-01 00:15:15 UTC
Tony: I took your patch and altered it to use a separate EpiphanyDownloadsSource, as well as splitting out Mozilla downloads in a similar way.  I just committed this to SVN. Please check to make sure I haven't broken anything.  

Thanks for this awesome patch!