On Wed, 16 Feb 2011 18:51:43 +0900 Jihoon Kim <[email protected]> said:

as some of the comments so far - this i think is beginning to overstep the mark
of what should be inside the toolkit. well sure - its enableable - but now
everyone has to go enabling or disabling it per entry - that's not good/ this
should likely be done as a module - some kind of input augmentation module that
can implement such policies. though the problem here is that edje actually
handled the nuts and bolts here and elm is at arms length. but there WILL be
the need for more policy like this when on small keyboards or virtual keyboards
- not just double space, but many other things (auto capitalization of first
letter in sentence).

now... should this be done in the entry .. or in the input method. as such i
actually think this is to be done in the input method - like a keyboard does
prediction, auto-correction and other things - this "auto correct 2x space to
'.'" is a function of the same mechanism that does the correction, preidction
and for that matter kanji conversion (japanese) and maybe pinyin to chinese
chars etc. etc. etc. - it's all the same thing and the others are right. this
is better done there - in fact right now i can't think of anything you'd need
for such input augmentation for devices that the IME subsystem can't handle.
yes - this means you need an IME with correction/prediction and alteration for
english too... as well as japanese, chinese, korean and more, but it's the
right way.

so right now - i'll have to say no to this patch, unless you can convince me
otherwise :)

> Dear all EFL developers,
> 
> This is patch for supporting autoperiod feature in entry widget.
> Autoperiod is the feature that period (.) will be automatically added 
> when space bar is pressed twice in a short time.
> 
> As you know, this feature can be seen in the blackberry, iphone, and so on.
> 
> You can test this feature according to the below step:
> 1. run elementary_test
> 2. Click 'Entry 5'
> 3. Click 'Autoperiod' button.
> 4. Type 'I love EFL'
> 5. Press space twice in a short time (within 0.6sec)
> 
> Would you please review this patch and apply in svn if it is ok?
> 
> Thanks.


-- 
------------- Codito, ergo sum - "I code, therefore I am" --------------
The Rasterman (Carsten Haitzler)    [email protected]


------------------------------------------------------------------------------
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to