<div dir="auto">It would be nice to have all global keyboard shortcuts in a single place. In my window manager I was overwriting a single method in order to hook keyboard and redefine shortcuts the way I like them (but it is broken now, it crashes, I need to make it work again with the current image).</div><div dir="auto"><br></div><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 19 Dec 2020 at 7:01 AM, Mauro Rizzi via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st">cuis-dev@lists.cuis.st</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)"><div dir="ltr"><div>Hello Everyone! <br></div><div><br></div><div>I just realized the Cuis-University image is outdated and contains packages that the current dev-cuis build doesn't (like the ToolFinder, which I think belongs on the base image from an usability side but i'm not going to die on that hill).</div><div><br></div><div>I made a new changeset to replace my previous collaboration with this issue that takes into account the current implementations in the keyboardEvent class to avoid reverting code and implementing package methods in the base image.</div><div><br></div><div>I just want to be clear that this changeset only contains the modification to the keyboardEvent class.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El vie, 18 dic 2020 a las 19:56, Franco Lucchesi (<<a href="mailto:lucchesifd@gmail.com" target="_blank">lucchesifd@gmail.com</a>>) escribió:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)"><div dir="ltr">Hello, sorry for the late reply, I ended up getting busy with real life matters and couldn't dedicate any time to looking at your fix.<br><br>I implemented it and reverted my Morph modification as you asked for. As a small sidenote, if you notice any discrepancies in the changeset format it was my fault, I couldn't find any way to merge them together inside of Cuis so I ended up manually appending it to the original changeset with an external text editor.<div><br></div><div>This however leads to a problem I'm not sure how to fix: this changeset appears to modify a package related message: sentTo:localPosition: from Tools-Finder, which doesn't feel correct to me. I thought about recategorizing this message back to "dispatching" but I'm not too sure if that would be correct either, I'll send both changesets: one that recategorizes the message to "dispatching" and one that leaves the category as is.</div><div><br></div><div><div><div>Would it be reasonable to leave this dependency category in the changeset? Or is it better to recategorize back to "dispatching"? Or would it be better to do a different solution with a different message?<br><br>Greetings, and happy holidays.<font style="color:rgb(136,136,136)"><br>-Franco</font></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El sáb, 12 dic 2020 a las 21:47, Mauro Rizzi (<<a href="mailto:mrizzi@fi.uba.ar" target="_blank">mrizzi@fi.uba.ar</a>>) escribió:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)"><div dir="ltr"><div>Hello Franco,</div><div><br></div><div>I looked at your implementation and the issue lies on the method you're using to intercept the ALT+ENTER command.</div><div><br></div><div>The attached changeset implements it in the KeyboardEvent class and it allows you to avoid sending the input to the morph that currently has focus.</div><div><br></div><div>Make sure to revert the change to focusKeyboardFor: and share the complete changeset with us!<br></div><div><br></div><div>Cheers!</div><div><b>Mauro Rizzi.</b><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El sáb, 12 dic 2020 a las 21:10, Franco Lucchesi via Cuis-dev (<<a href="mailto:cuis-dev@lists.cuis.st" target="_blank">cuis-dev@lists.cuis.st</a>>) escribió:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)"><div dir="ltr"><div>Hello, recently I've been working with Cuis Smalltalk and whilst it's been a great experience the lack of an easy full screen toggle was starting to annoy me, so I decided to examine some of the Display related code and implement it myself.</div><div><br></div>As the topic says, all this changeset does is add Alt+Return as a shortcut to toggle full screen on or off, it also additionally does a small refactor on some other full screen related messages.<div><br><div>However, it has one bug: if you do Alt+Return whilst you are editing a textbox the full screen procedure seems to abort and instead an unusual character gets written in the current textbox, I haven't been able to figure out how to fix it but my best guess is to deselect everything before going full screen, which I also couldn't figure out how to do.</div><div><br></div><div>This is my first time writing to a mail list and contributing updates to open source software so please let me know if I have done anything wrong in the process.</div><div><div><br></div><div>Kind regards, and have a good weekend.</div><div><br></div><div>-Franco</div></div></div></div>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
</blockquote></div>
</blockquote></div>
</blockquote></div>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
</blockquote></div></div>