<div dir="ltr">Without answering Christoph's question, I'd like to ask an additional one about this topic:   I think it's wasteful and complicated to use so many different keys for removing the bracket layer.  I liked this idea of separate closer command at first but, after using it a bit, I don't.  Now, not only does the user have to think about the different keys for "adding vs. removing", they also have to consider *which* closer bracket to press -- e.g., close bracket?  close brace?  close paren? -- even though the context makes that burden unnecessary for the user, lest they should press the wrong one and be disrupted even more.  It really does force me to "pause" where, with toggle, it didn't.<div><br></div><div>It also means we now consume... 4(?!) hot keys for a function that used to only consume 1.<div><br></div><div>If there's no way to go back to toggle, we should at least consume only one hot-key for "removing a level" -- either ) or ] -- that will work on any of the three.</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 27, 2022 at 4:10 PM <<a href="mailto:christoph.thiede@student.hpi.uni-potsdam.de" target="_blank">christoph.thiede@student.hpi.uni-potsdam.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Jaromir, Marcel, all,<br>
<br>
with Morphic-mt.1824 in current Trunk, we have the following behavior for closer brackets:<br>
<br>
<i>> Via "enclose selection" preference, an opening bracket adds a level and a closer bracket removes a level. No toggling anymore.<br>
</i><br>
So pressing a closer bracket removes the pair of brackets that is <i>*enclosing*</i> the selection. However, I still wonder whether we should also remove the pair of brackets <i>*inside*</i> the selection if there is no further pair of brackets.<br>
<br>
So that<br>
<br>
    <b><font color="#008000">[123]</font></b> (entire line selected)<br>
<br>
when pressing<br>
<br>
    ]<br>
<br>
becomes<br>
<br>
    <b><font color="#008000">123</font></b> (entire line selected)<br>
<br>
instead of (as in current Trunk)<br>
<br>
    ] (nothing selected)<br>
<br>
Or would this be more inconsistent than convenient? Wdyt?<br>
<br>
Best,<br>
Christoph<br>
<br>
<font color="#808080">---<br>
</font><font color="#808080"><i>Sent from </i></font><font color="#808080"><i><a href="https://github.com/hpi-swa-lab/squeak-inbox-talk" target="_blank"><u><font color="#808080">Squeak Inbox Talk</font></u></a></i></font><br>
<br>
On 2022-02-07T23:33:53+01:00, <a href="mailto:christoph.thiede@student.hpi.uni-potsdam.de" target="_blank">christoph.thiede@student.hpi.uni-potsdam.de</a> wrote:<br>
<br>
> Just a quick note to myself, compensating for the lack of meta information on the list:<br>
> This issue has been resolved via Morphic-mt.1809. Thanks. :-)<br>
> <br>
> Best,<br>
> Christoph<br>
> <br>
> ---<br>
> Sent from Squeak Inbox Talk<br>
> <br>
> On 2021-11-01T22:09:46+01:00, christoph.thiede at <a href="http://student.hpi.uni-potsdam.de" target="_blank">student.hpi.uni-potsdam.de</a> wrote:<br>
> <br>
> > Hi all!<br>
> > <br>
> > This changeset refines the existing autoEnclose mechanism. Instead of having inserted enclosing brackets always, you can now activate a new preference to only insert these characters if there is any space after the cursor. This matches VS Code's setting value "beforeWhitespace" for "editor.autoClosingBrackets".<br>
> > <br>
> > To me, this mode feels much more convenient because when having traditional autoEnclose enabled, I have been getting angry again and again when I wanted to bracketize an existing expression in some code and suddenly was disrupted by an unneeded closing bracket.<br>
> > <br>
> > The changeset does not manipulate any defaults. The PreferenceWizard and the ReleaseBuilder are updated, too. Please honor the postscript of the changeset when merging it.<br>
> > <br>
> > Best,<br>
> > Christoph<br>
> > <br>
> > ---<br>
> > Sent from Squeak Inbox Talk<br>
> > -------------- next part --------------<br>
> > An HTML attachment was scrubbed...<br>
> > URL: <<a href="http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20211101/4e337f68/attachment.html" target="_blank">http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20211101/4e337f68/attachment.html</a>><br>
> > -------------- next part --------------<br>
> > A non-text attachment was scrubbed...<br>
> > Name: autoEncloseBeforeSpace.2.cs<br>
> > Type: application/octet-stream<br>
> > Size: 13012 bytes<br>
> > Desc: not available<br>
> > URL: <<a href="http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20211101/4e337f68/attachment.obj" target="_blank">http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20211101/4e337f68/attachment.obj</a>><br>
> > <br>
> > <br>
> -------------- next part --------------<br>
> An HTML attachment was scrubbed...<br>
> URL: <<a href="http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20220207/34fe035a/attachment.html" target="_blank">http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20220207/34fe035a/attachment.html</a>><br>
> <br>
> <br>
</blockquote></div>