Hi Marcel,<br>
<br>
there is another limitation of this heuristic because it is tied to the nesting depth of widgets. :-) For instance, the MessageNames tool has the wrong defaultFocus because the text morph in the search bar on the left is nested into a panel whereas the list on the right is not nested. Would you agree to extend the heuristic to fix that? :-)<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"><u><font color="#808080">Squeak Inbox Talk</font></u></a></i></font><br>
<br>
On 2021-11-23T10:43:39+01:00, marcel.taeumel@hpi.de wrote:<br>
<br>
> Okay. See Morphic-mt.1799 in Trunk.<br>
> <br>
> Best,<br>
> Marcel<br>
> <br>
> Am 22.11.2021 19:22:37 schrieb Thiede, Christoph <christoph.thiede at student.hpi.uni-potsdam.de>:<br>
> Hi Marcel,<br>
> <br>
> thanks again for reviewing and merging this!<br>
> <br>
> > Adds and uses breadth-first search for submorph lookup because it matches the natural way widgets are organized in a window. The existing depth-first in #allMorphsDo: might take surprising paths.<br>
> <br>
> I keep finding some tools where this heuristic is not yet good enough. For example, in the Monticello Browser, I would expect the package list on the left to be focused by default, but instead, the repository list on the right is focused. What would you think about a strategy that orders all descendant morphs by their natural reading order and always chooses the left-most from the first row (#x ascending , #y ascending)? Or would you prefer to specify this information in the relevant toolbuilder methods instead?<br>
> <br>
> Best,<br>
> Christoph<br>
> Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel<br>
> Gesendet: Freitag, 19. November 2021 18:17:25<br>
> An: squeak-dev<br>
> Betreff: Re: [squeak-dev] Merge Request: defaultFocusMorph.cs<br>
>  <br>
> Merged.<br>
> <br>
> Revisions in SystemWindow >> #defaultFocusMorph and more tool-specific use of #defaultFocus: besides the preference browser.<br>
> Am 20.10.2021 16:31:00 schrieb christoph.thiede at student.hpi.uni-potsdam.de <christoph.thiede at student.hpi.uni-potsdam.de>:<br>
> Hi all,<br>
> <br>
> if a new window is invoked via a keyboard gesture, with the proposed changeset, the keyboard focus is automatically passed to the new window. See SystemWindow >> #defaultFocusMorph. By default, the first keyboard-interested (sub)+morph is selected, but you can specify a different focus holder by setting the #defaultFocusMorph property of the window or the #defaultFocus parameter of the pluggable window spec.<br>
> <br>
> I think this improves the keyboard workflow in Squeak a bit. For instance, it enables the following workflows:<br>
> - Ctrl + 0 (search bar), type '6 * 7', Alt + o (spawn), Alt + p (print it)<br>
> - Ctrl + 3 (Tools menu), type 'browser', Enter, Alt + f (find), type 'Object', Enter<br>
> <br>
> Please review and merge if you don't have any objections. Happy squeaking! :-)<br>
> <br>
> Best,<br>
> Christoph<br>
> <br>
> ---<br>
> Sent from Squeak Inbox Talk [https://github.com/hpi-swa-lab/squeak-inbox-talk]<br>
> -------------- next part --------------<br>
> An HTML attachment was scrubbed...<br>
> URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20211123/72ca412e/attachment.html><br>
> <br>
>