GNOME Bugzilla – Bug 685899
Smileys are always inserted at the end of the entry, not at the cursor position
Last modified: 2013-11-17 19:19:55 UTC
The cursor in the following description is “|”. 1. Open a conversation window: entry shows “|” 2. Type some text: entry shows “hello|” 3. Move the cursor to the beginning of the line: “|hello” 4. Right-click the entry, choose Insert Smiley → :-(|). Expected result: a monkey is inserted where the cursor is, before the word “hello”. Actual result: a monkey is inserted at the end of the entry.
Created attachment 259585 [details] [review] Insert smileys at the cursor position instead of the end of entry
Looks good to me. Cassidy, could you please check?
Review of attachment 259585 [details] [review]: This fixes the right-click menu case, but it doesn't fix picking that option in the Conversation menu at the top of the window. The code for that lives in src/empathy-chat-window.c . It would be good to add an empathy_chat_insert_smiley() method to libempathy-gtk/empathy-chat.c and call it from src/empathy-chat-window.c rather than duplicating the logic!
Created attachment 259873 [details] [review] Insert smileys at the cursor position instead of the end of entry Updated patch.
Looks good to me now. Should I apply it locally and commit with my credentials? I am new at committing patches for others.
veena, check: https://git.gnome.org/browse/empathy/commit/?id=25b40bf1b21a7c73a5b87649e3fb3536d64d0a97 Your submission had trailing whitespaces at line 32. I have corrected that before committing. Be careful not to include those in your future submissions. This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.