how can you test/use it ? On Wed, Apr 14, 2010 at 2:51 PM, Stéphane Ducasse <stephane.ducasse@inria.fr

wrote:

[update 1.1] #11315 11315 -----

- Issue 1225: Harvest textEditor from Squeak. Thanks Fernando. Once more now with the textEditor Classes. Sorry for the mess

Now we should debug it.

Stef On Apr 14, 2010, at 2:25 PM, Lukas Renggli wrote:

...

This is so cool. I will give a try.

Thanks Fernando, Juan and Stef for making this possible.

Lukas

On 14 April 2010 14:21, Stéphane Ducasse <stephane.ducasse@inria.fr> wrote:

...

11314 -----

- Merge gofer - Issue 1225: Harvest textEditor from Squeak. Thanks Fernando.

Now we should debug it. So please use them and we will find and fix the bugs.

Stef

_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

-- Lukas Renggli www.lukas-renggli.ch

_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Also most of the bugs happen when trying to use the Cuis old code for system navigation. SmalltalkTextEditor methods for browsing senders, implementors and alike. Stef fixed a couple of them, today i fixed a couple more. Tomorrow i'll post a slice with the fixes i've done this past week. Fernando On Apr 14, 2010, at 9:06 PM, Fernando olivero wrote:

...

...

...

Now we should debug it. So please use them and we will find and fix the bugs.

Stef

_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

On Apr 15, 2010, at 12:43 43AM, Fernando olivero wrote:

Also most of the bugs happen when trying to use the Cuis old code for system navigation.

SmalltalkTextEditor methods for browsing senders, implementors and alike.

Stef fixed a couple of them, today i fixed a couple more.

Tomorrow i'll post a slice with the fixes i've done this past week.

Fernando I went through the beginning of various keyboard shortcuts in TextEditor, the following were my notes:

End (#cursorEnd:) raises mnu Cmd-h (#setSearchString:) raised mnu - Ctrl-Enter (#crWithIndent:) is the same as default enter-action. Auto-indent setting is no longer available. - Esc action defined in Editor shortcut will never be used, since NewTextMorph/EntryFieldMorph returns early in keyStroke: method for esc characters (To do Announcement, which only morph itself is registered to). Should these Announcements be located in Editor, or its morph? If in editor, example1 could still work, you'd just do e model onEscapeSend: #delete to: e. Or better yet: e model announcer on: EscapePressed send: #delete to: e. that way you don't have duplicate utility methods for every announcement class / handling method you want to be able to plug into. Not to mention you can use the same Editor / Announcement plugin method for other frameworks than morphic. Also, SimpleEditor>>dispatchOn: has special code for Character cr on top, but the TextEditor code for handling of cr below (the asciiValue = 13 part) has not been removed although it is never called. In regards to the cr-handling in dispatchOn:, I don't really see the value of allowing typing of multiple line ending conventions in strings inside the image using modifier keys. Re: the previous lengthy discussion on the topic, I'm with the side that wanted to have consistent line endings in in-image strings. There are also references to Preferences 9 or so places in the code, these should be changed to Settings-equivalents. Cheers, Henry