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 621516 - More maps integration in addressbooks and calendars: Calendar events on maps
More maps integration in addressbooks and calendars: Calendar events on maps
Status: RESOLVED OBSOLETE
Product: evolution
Classification: Applications
Component: Plugins
3.4.x (obsolete)
Other All
: Normal enhancement
: ---
Assigned To: evolution-plugin-maintainers
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2010-06-14 09:39 UTC by Cedric Bosdonnat
Modified: 2021-05-19 11:38 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
A patch to implement the features (53.78 KB, patch)
2010-06-14 09:39 UTC, Cedric Bosdonnat
needs-work Details | Review

Description Cedric Bosdonnat 2010-06-14 09:39:57 UTC
Created attachment 163562 [details] [review]
A patch to implement the features

Some nice feature to add:
 * Show calendar events on maps
 * Some more integration with contacts

The attached patch is adding those features but contain some tiny problems to fix:
 * the address parsed in the calendar event location can only be US-formatted addresses. Some work is ongoing on that point.
 * the focus chain of the new location details dialog isn't correct
Comment 1 Dan Vrátil 2011-04-20 11:23:47 UTC
I've noticed  that you are using synchronous method to resolve GPS coordinates from address, which is definitely not OK (this applies for the original contact-maps plugin too - try to open map for 200 contacts with a slow connection). Use  geoclue_geocode_address_to_position_async() instead.

Since you are integrating the map feature directly into the Evolution on some places, I think it would make sense, to merge this with bug 642557, creating a common function to resolve GPS coordinates and put markers on widget which would return prepared champlain widget and then just call it from various places in Evo.

I would definitely merge the "all contacts map" and the event map with the bug 642557, but I'm not sure abou the single contact map, since the other bug provides the same directly within the contacts preview, which I think is a better feature ;) Have you tested the single full-width map as we discussed on IRC yet?
Comment 2 André Klapper 2012-02-02 18:04:09 UTC
Cedric: Are you still around and could answer Dan's comments here?
Comment 3 Cedric Bosdonnat 2012-02-02 19:06:23 UTC
André, Dan,

I'm still around, though I don't have time for evolution these days. Feel free to do whatever is needed with that patch. I didn't have time to test the single full-width map yet... do it as you like it would be already better than nothing ;)
Comment 4 André Klapper 2021-05-19 11:38:04 UTC
GNOME is going to shut down bugzilla.gnome.org in favor of gitlab.gnome.org. 
As part of that, we are mass-closing older open tickets in bugzilla.gnome.org
which have not seen updates for a longer time (resources are unfortunately
quite limited so not every ticket can get handled).

If you can still reproduce the situation described in this ticket in a recent
and supported software version, then please follow
  https://wiki.gnome.org/Community/GettingInTouch/BugReportingGuidelines
and create a new enhancement request ticket at
  https://gitlab.gnome.org/GNOME/evolution/-/issues/

Thank you for your understanding and your help.