[squeak-dev] Proof-of-concept: moving Smalltalk code editing support into SmalltalkEditor

Tim Johnson digit at sonic.net
Sat Mar 24 19:04:02 UTC 2018


Hi all,

I went on a journey this morning which could have been quixotic, but I 
believe I actually found and slayed a small windmill.

I was a bit annoyed by how autoEnclose (and by extension autoIndent) 
were in effect for even the most basic of TextMorphs, e.g. those dragged 
out of the Supplies flap. I was also kind of disappointed that turning 
on/off the auto-enclose preference affected all text morphs, even those 
not specifically designated for Smalltalk code editing.  I was 
emboldened when I saw the class comment for TextEditor: "[I] have no 
specific facilities for editing Smalltalk code. Those are found in 
SmalltalkEditor."

I found myself able to address this by making a few changes:

* moving autoEnclose and autoIndent functionality out of TextEditor and 
into SmalltalkEditor

* changing the default editor class for TextMorph to TextEditor

* specifying that the #editorClass for TextMorphForEditView should be a 
SmalltalkEditor

* changing PreferenceWizard to recognize these preferences have a new 
home

I'm attaching a changeset to this email and not uploading an .mcz to 
inbox (sorry).  This is because I don't know how to represent changes 
across many projects in Monticello, and I sort of consider this a 
proof-of-concept rather than something I think warrants immediate 
inclusion in the image.

Please do let me know what you think about this idea.

I also fear my implementation of 
SmalltalkEditor>>#dispatchOnKeyboardEvent: might be lacking.

Thanks,
Tim

ps - I think a next step of moving Smalltalk editing support into 
SmalltalkEditor might be to move the call to 
ToolSet>>#codeCompletionAround:textMorph:keyStroke: out of 
TextMorph>>#keyStroke: and into its subclass, in 
TextMorphForEditView>>#keyStroke:.  I didn't complete that move and did 
not quite figure out how the subclass version of that method works.
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: MoveUpAutoIndentAndEnclose.1.cs
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20180324/1146c701/attachment.ksh>


More information about the Squeak-dev mailing list