GNOME Bugzilla – Bug 563867
"Open Link in Browser" uses "&" instead of &
Last modified: 2009-02-03 13:45:11 UTC
If you have a mail with contains a link containing more than one query (e.g. http://www.google.de/search?hl=de&q=Evolution ) and do a right click on that link and select "Open Link in Browser" it will send ...&q=... to the browser. I found some bug reports stating that this was fixed in GtkHTML (which is right as a simple left click works as espected), but it looks like it was forgotten in the right click menu.
Do you happen to have the GtkHTML bug number handy? The "Open Link in Browser" part lives in Evolution, so the same fix probably just needs to be applied there.
This is the one that resulted in a fix for GtkHTML: bug #485700 But there is also this one which seems to include the unescaping code into Evo already (for Copy Link): bug #506948
Created attachment 125953 [details] [review] use present unescaping function for both This makes the already present unescaping function available to both functions. em-utils looked like the right place for the utility function to me, not fully sure though (evo is quite big).
Commit to stable/trunk
Sorry, there have been some changes in the source code and your patch probably needs to be re-based/modified. Marking as such.
Created attachment 126263 [details] [review] updated patch (for trunk) Okay, this should apply to trunk again. The only thing that needed to be changed, was the em-popup.c chunk. Attachment 125953 [details] theoretically still applies to the 2.24 series. But I don't know if it's still needed as it looks like I missed the 2.24.3 release.
(In reply to comment #3) > Created an attachment (id=125953) [edit] > use present unescaping function for both Patch committed to stable (gnome-2-24) branch as r37062 http://svn.gnome.org/viewvc/evolution?view=revision&revision=37062 (In reply to comment #6) > Created an attachment (id=126263) [edit] > updated patch (for trunk) Patch committed to SVN trunk as r37063 http://svn.gnome.org/viewvc/evolution?view=revision&revision=37063 > Attachment 125953 [details] [edit] theoretically still applies to the 2.24 series. But I don't > know if it's still needed as it looks like I missed the 2.24.3 release. I think a 2.24.4 is planned. Your patch will make it to that release :-)
Just got Evo 2.24.4 through my package manager. I guess this can be closed then.