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 552296 - Autocompletion on contact names for "new message" dialog box
Autocompletion on contact names for "new message" dialog box
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: Contact List
2.23.x
Other All
: Normal enhancement
: ---
Assigned To: empathy-maint
Depends on:
Blocks:
 
 
Reported: 2008-09-15 00:45 UTC by chantra
Modified: 2008-12-09 09:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
enabling autocompletion patch (5.63 KB, patch)
2008-09-15 00:52 UTC, chantra
reviewed Details | Review
new autocompletion patch (9.40 KB, patch)
2008-12-03 09:48 UTC, chantra
reviewed Details | Review

Description chantra 2008-09-15 00:45:43 UTC
Currently, when one wants to send a new message using the "new message" dialog, the contactID needs to be entered, which is not the most user friendly.
Based on
http://git.collabora.co.uk/?p=empathy.git;a=commit;h=1304e07b0f56c86f4c9182a25b5f4d584bb47c13

I have commited a branch to http://github.com/chantra/empathy_new_msg_completion/tree/master which makes new message easier to use by enabling auto completion on the contact name.
Comment 1 chantra 2008-09-15 00:52:55 UTC
Created attachment 118719 [details] [review]
enabling autocompletion patch
Comment 2 chantra 2008-09-15 17:39:58 UTC
some codig style changes, branch is now at http://github.com/chantra/empathy/tree/new-msg-completion
Comment 3 Xavier Claessens 2008-11-29 22:02:53 UTC
Sorry to not reply earlier, I looked quickly at your branch. Could you please fix the coding style? We are using tab indentation: http://developer.imendio.com/codingstyle

Thanks :)
Comment 4 chantra 2008-11-30 12:01:09 UTC
I will change my indent tab from 2 to 8 :) and ping you back once done.
Comment 5 chantra 2008-12-03 09:48:24 UTC
Created attachment 123858 [details] [review]
new autocompletion patch

diff against commit 460e2e92a9b9861ebbc5cc318abc9ad57fa4c801 (Mon Dec 1 12:46:27 2008 +0000)
Comment 6 Xavier Claessens 2008-12-07 22:43:48 UTC
I cleaned your code and merged. About coding style: I changed indentation from 8 spaces to 1 tab, added one space between function name and bracket and cut lines to fit in 80char.

Thanks you for your patch!
Comment 7 chantra 2008-12-09 09:39:49 UTC
Hi Xavier,

thanks for that, and sorry I did not make a neater patch.