Re: [E-devel] [PATCH] Patch to keep the cursor position regardless of changing edj

2011-03-17 Thread Jihoon Kim
n' Cc: enlightenment-devel@lists.sourceforge.net Subject: Re: [E-devel] [PATCH] Patch to keep the cursor position regardless of changing edj There was some typo in my previous e-mail. The API that I mentioned in the paragraph 1 is 'edje_object_message_signal_process' NOT 

Re: [E-devel] [PATCH] Patch to keep the cursor position regardless of changing edj

2011-03-17 Thread Tom Hacohen
-Original Message- > From: Jihoon Kim [mailto:jihoon48@samsung.com] > Sent: Thursday, March 17, 2011 2:39 PM > To: 'Tom Hacohen' > Cc: enlightenment-devel@lists.sourceforge.net > Subject: Re: [E-devel] [PATCH] Patch to keep the cursor position regardless &g

Re: [E-devel] [PATCH] Patch to keep the cursor position regardless of changing edj

2011-03-16 Thread Jihoon Kim
PM To: 'Tom Hacohen' Cc: 'enlightenment-devel@lists.sourceforge.net' Subject: RE: [E-devel] [PATCH] Patch to keep the cursor position regardless of changing edj 1. The reason why I call edje_object_signal_emit is that _signal_cursor_changed function was not called in the below

Re: [E-devel] [PATCH] Patch to keep the cursor position regardless of changing edj

2011-03-16 Thread Jihoon Kim
al Message- From: Tom Hacohen [mailto:tom.haco...@partner.samsung.com] Sent: Thursday, March 17, 2011 1:29 AM To: Jihoon Kim Cc: enlightenment-devel@lists.sourceforge.net Subject: Re: [E-devel] [PATCH] Patch to keep the cursor position regardless of changing edj Dear Jihoon, Thanks a lot for the pat

Re: [E-devel] [PATCH] Patch to keep the cursor position regardless of changing edj

2011-03-16 Thread Tom Hacohen
Dear Jihoon, Thanks a lot for the patch, I have one question and one comment: Question: In line 89 of the patch, you added: edje_object_message_signal_process(wd->ent); Why is it needed there? Comment: I think all the + elm_entry_cursor_pos_set(obj, wd->cursor_pos); + if (elm_widget_focus_g

[E-devel] [PATCH] Patch to keep the cursor position regardless of changing edj

2011-03-16 Thread Jihoon Kim
Hello. Whenever the theme (edj) is changed in entry, the cursor position in entry is reset because elm_entry does not save the cursor position in the entry. This patch is for keeping the cursor position in entry regardless of changing theme (edj). It will save the cursor position in entry and set