On Mon, 21 Feb 2011 12:03:42 +0900 WooHyun Jung <wh0705.j...@samsung.com> said:
in svn! thanks! :) > I made a patch - just using the position of "old-cursor". > Is it enough for this problem ? > > > -----Original Message----- > From: Carsten Haitzler (The Rasterman) [mailto:ras...@rasterman.com] > Sent: Sunday, February 20, 2011 11:07 PM > To: Tom Hacohen > Cc: WooHyun Jung; enlightenment-devel@lists.sourceforge.net > Subject: Re: [E-devel] One bug in _edje_key_down_cb func (in edje_entry.c) > > On Sat, 19 Feb 2011 12:02:10 +0200 Tom Hacohen <t...@stosb.com> said: > > > On Sat, Feb 19, 2011 at 2:54 AM, Carsten Haitzler > <ras...@rasterman.com>wrote: > > > > > no no - you got him wrong. currently is creates a tmp cursor, copies > > > current cursor to tmp cursor, then does the operation (delete char > > > to the left of cursor). it then COMPARES cursor and tmp cursor. if > > > different - send messages to say the cursor moved/changed. entry > > > depends on this to know if/when to reposition things that are > > > relative to the cursor. the cursor DID change BUT... > > > textblock adjusted BOTH cursors so they are now the same so the > > > comparison doesnt show a difference, thus no emit of signal. > > > > > > > > That's what I thought he meant (just re-read my previous email and > > noticed I didn't explain myself very well). > > > > WooHyun: regarding your issue: I have another solution, you can use > > evas_textblock_cursor_pos_get to get the cursor position as a number > > which obviously will not get updated when you delete, so your > > comparison will still remain valid. Currently this function is pretty > > fast, but if you ever think it's not fast enough, I can easily make it > > WAY faster. > > > > I have one more question: What do you expect to happen when a user > > presses delete? > > actually he means backspace. what is meant to happen is elm is meant to get > a signal that the entry cursor changed position - which it did :) getting > pos before the op, pos after and comparing - if different, then send signal > will work. it just needs fixing :) > > > In some cases, the coordinates of the cursor will change, in some > > cases they won't, so I'm still not sure you want the actual position. > > > > What are you using this signal for? Maybe we'll have to fix something > there. > > > > -- > > Tom. > > > -- > ------------- Codito, ergo sum - "I code, therefore I am" -------------- > The Rasterman (Carsten Haitzler) ras...@rasterman.com -- ------------- Codito, ergo sum - "I code, therefore I am" -------------- The Rasterman (Carsten Haitzler) ras...@rasterman.com ------------------------------------------------------------------------------ The ultimate all-in-one performance toolkit: Intel(R) Parallel Studio XE: Pinpoint memory and threading errors before they happen. Find and fix more than 250 security defects in the development cycle. Locate bottlenecks in serial and parallel code that limit performance. http://p.sf.net/sfu/intel-dev2devfeb _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
