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 371705 - Calendar's day view does not reset its IM context
Calendar's day view does not reset its IM context
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Calendar
3.0.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: evolution-calendar-maintainers
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2006-11-06 20:42 UTC by Matthew Barnes
Modified: 2013-09-13 01:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch. (4.93 KB, patch)
2006-12-18 12:54 UTC, makuchaku (Mayank)
none Details | Review
Updated patch (4.93 KB, patch)
2006-12-20 10:43 UTC, makuchaku (Mayank)
needs-work Details | Review
evo patch (6.66 KB, patch)
2010-11-23 14:14 UTC, Milan Crha
committed Details | Review

Description Matthew Barnes 2006-11-06 20:42:38 UTC
Forwarding this from a downstream bug report:
http://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=176400
Verified this still exists in Evolution 2.8/2.9.


Description of problem:
Tested evolution calendar with scim and noticed that the pre-edit buffer is not
reset when scim is activated in another time slot (input context). As a result,
previous entries gets replicated.

Version-Release number of selected component (if applicable):
evolution-data-server-1.5.2-1.1

How reproducible:
Always

Steps to Reproduce:
1.start evolution -> calendar
2.in day view, click on 1PM
3.enter a to get focus
4.ctrl-space to activate scim
5.enter 'aka'
6.click on the next time slot and enter a
7.ctrl space to activate scim and enter genki
  
Actual results:
ããããã

Expected results:
ããã

Additional info:
Comment 1 Matthew Barnes 2006-11-06 20:52:25 UTC
Our solution to this particular problem was to call gtk_im_context_reset() from e_day_view_start_editing_event().  But it's a poor hack.

EText should reset its own input method context at the appropriate times.  It looks like some progress was made towards this (see e_text_reset_im_context), but the code is unfinished.
Comment 2 makuchaku (Mayank) 2006-12-18 12:54:04 UTC
Created attachment 78560 [details] [review]
Proposed patch.

Hi,
The patch is up for review.

Thanks,
Mayank
Comment 3 makuchaku (Mayank) 2006-12-20 10:43:26 UTC
Created attachment 78680 [details] [review]
Updated patch

Patch for review.
Comment 4 Chenthill P 2009-01-21 07:06:52 UTC
mbanes, it would be better if you review this patch..
Comment 5 Chenthill P 2009-11-30 07:20:13 UTC
mbarnes, ping :)
Comment 6 Milan Crha 2010-11-16 15:18:56 UTC
Patch doesn't work. My steps are:
a) go to day view
b) create event A and Event B
c) click on the event A to edit its summary inline
d) ctrl+space to active IM and write "hong" (without quotes)
e) click on event B a press space, the character is removed from B and
   is pasted to A.

With the patch is not moved the text from one event to another, but it's also not kept there, and turning on the IM again is not possible after moving to A.
Comment 7 Milan Crha 2010-11-23 14:14:29 UTC
Created attachment 175110 [details] [review]
evo patch

for evolution;

Slightly extended previous patch. Unless the IM is finished, the text is not used.
Comment 8 Milan Crha 2010-11-23 14:27:56 UTC
Created commit e490db5 in evo master (2.91.3+)