Done:<br>
https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/42#issuecomment-947689830<br>
This version can be moved to treated.<br>
<br>
Best,<br>
Christoph<br>
<br>
<font color="#808080">---<br>
</font><i><font color="#808080">Sent from </font></i><i><u><a href="https://github.com/hpi-swa-lab/squeak-inbox-talk"><font color="#808080">Squeak Inbox Talk</font></a></u></i><br>
<br>
On 2021-10-20T09:08:06+02:00, marcel.taeumel@hpi.de wrote:<br>
<br>
> There are open input event issues on GitHub. Maybe add a comment there or open a new ticket.<br>
> <br>
> Best,<br>
> Marcel<br>
> Am 19.10.2021 19:39:18 schrieb Thiede, Christoph <christoph.thiede at student.hpi.uni-potsdam.de>:<br>
> Hi Marcel,<br>
> <br>
> alright, this looks indeed like a bug in the (Windows) VM. Hm ... Is there any central place where we are tracking open keyboard handling problems? Maybe we should collect them in an OSVM issue? :-)<br>
> <br>
> Best,<br>
> Christoph<br>
> Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel<br>
> Gesendet: Dienstag, 19. Oktober 2021 15:36:08<br>
> An: squeak-dev<br>
> Betreff: Re: [squeak-dev] The Inbox: Morphic-ct.1786.mcz<br>
>  <br>
> Note that shortcuts implemented via EventKeyChar (or #keyStroke:) use #keyCharacter and should therefore not be combined with #shiftPressed since the SHIFT-key actually modifiers the character delivered. Just use #keyCharacter and the other modifiers (e.g., #controlPressed or #commandPressed).<br>
> <br>
> In this example, SHIFT seems to be ignored in the VM when CTRL is pressed. SHIFT+1 yields $! but CTRL+SHIFT+1 yields $1. This is clearly a bug in the VM.<br>
> <br>
> So, I would rather not make the change as proposed here.<br>
> <br>
> Best,<br>
> Marcel<br>
> <br>
> P.S.: Note that this is not the minimal change you could have made to this method to propose your fix. By merging the two checks into a more complex condition, you impaired readability a little bit. At least through my eyes. I would have probably just added a check for #shiftPressed below #isKeystroke and also added some commentary on what the bug here is.<br>
> Am 18.10.2021 17:52:43 schrieb commits at source.squeak.org <commits at source.squeak.org>:<br>
> A new version of Morphic was added to project The Inbox:<br>
> http://source.squeak.org/inbox/Morphic-ct.1786.mcz<br>
> <br>
> ==================== Summary ====================<br>
> <br>
> Name: Morphic-ct.1786<br>
> Author: ct<br>
> Time: 18 October 2021, 5:52:25.059559 pm<br>
> UUID: 322e30d5-aa3f-cb45-843c-c01bf40a00a1<br>
> Ancestors: Morphic-ul.1780<br>
> <br>
> Do not invoke docking bar menu via Ctrl + Shift + . Keep this shortcut available for others morphs.<br>
> <br>
> =============== Diff against Morphic-ul.1780 ===============<br>
> <br>
> Item was changed:<br>
> ----- Method: DockingBarMorph>>filterEvent:for: (in category 'events-processing') -----<br>
> filterEvent: aKeyboardEvent for: anObject<br>
> "Provide keyboard shortcuts."<br>
> <br>
> | index itemToSelect |<br>
> + (aKeyboardEvent isKeystroke and: [aKeyboardEvent controlKeyPressed] and: [aKeyboardEvent shiftPressed not])<br>
> -<br>
> - aKeyboardEvent controlKeyPressed<br>
> ifFalse: [^ aKeyboardEvent].<br>
> +<br>
> -<br>
> - aKeyboardEvent isKeystroke<br>
> - ifFalse: [^ aKeyboardEvent].<br>
> -<br>
> "Search field."<br>
> aKeyboardEvent keyCharacter = $0<br>
> ifTrue: [<br>
> self searchBarMorph ifNotNil: [ :morph |<br>
> morph model activate: aKeyboardEvent in: morph ].<br>
> ^ aKeyboardEvent ignore "hit!!"].<br>
> <br>
> "Select menu items."<br>
> (aKeyboardEvent keyValue<br>
> between: $1 asciiValue<br>
> and: $9 asciiValue)<br>
> ifFalse: [^ aKeyboardEvent].<br>
> <br>
> index := aKeyboardEvent keyValue - $1 asciiValue + 1.<br>
> itemToSelect := (self submorphs select: [ :each |<br>
> each isMenuItemMorph ])<br>
> at: index<br>
> ifAbsent: [^ aKeyboardEvent].<br>
> +<br>
> -<br>
> self activate: aKeyboardEvent.<br>
> self<br>
> selectItem: itemToSelect<br>
> event: aKeyboardEvent.<br>
> +<br>
> -<br>
> ^ aKeyboardEvent ignore "hit!!"!<br>
> <br>
> <br>
> -------------- next part --------------<br>
> An HTML attachment was scrubbed...<br>
> URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20211020/a65c435e/attachment.html><br>
> <br>