Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not working on 5.1 #63

Open
143mailliw opened this issue Oct 9, 2016 · 5 comments
Open

Not working on 5.1 #63

143mailliw opened this issue Oct 9, 2016 · 5 comments
Assignees

Comments

@143mailliw
Copy link

No description provided.

@fawind fawind self-assigned this Oct 10, 2016
@fawind
Copy link
Contributor

fawind commented Oct 10, 2016

I will have a look at this in the next few days.

@fniephaus
Copy link
Member

Have you tried this one yet?
https://github.com/HPI-SWA-Teaching/SWT16-Project-06

We haven't merged the code into this repository yet.

@fawind
Copy link
Contributor

fawind commented Oct 12, 2016

I could find the following issue using Squeak 5.1:

  • The focus is not dropped when Algernon is closed and no item was searched. This results in a frozen world until Algernon is focused again. This also happens when activating Algernon using the settings menu.
  • The default keyboard shortcut Cmd+<Space> does not work anymore. I could not find the root cause here. Changing it to something else worked fine.

I opened a PR on the other repo which should fix these bugs: hpi-swa-teaching/Algernon-Launcher#28

@ComputeLinux: Were these the problem you experienced?

If you want to use Algernon on 5.1 immediately, I recommend you using the above repo. There you can set a new keyboard shortcut in the settings menu. Additionally you need to add the line self dropFocus: hand. in Algernon>ALGCore>deactivate:. Otherwise you can wait until the changes are merged.

@marceltaeumel
Copy link
Contributor

I recommend using event filters to trigger Algernon. ALGCore itself could act as a filter itself. For an example, see:

  • PasteUpMorph >> #addKeyboardShortcuts
  • PasteUpMorph >> #filterEvent:for:

Note that class objects can also be used as a filter. They only have to implement #filterEvent:for:.

If Algernon should act modally exclusive, work with the mouse focus. For an example, see:

  • MenuMorph >> #activate:
  • MenuMorph >> #mouseUp:

If is a good idea to keep track of the former mouse focus and keyboard focus holders and restore them once Algernon closes. With this, the world should never freeze. For your issue here, [CMD]+[.] should always revive event handling in the world.

For other issues with keyboard events, note that the recent macOS VMs have issues with correct key codes: OpenSmalltalk/opensmalltalk-vm#43

@fawind
Copy link
Contributor

fawind commented Oct 16, 2016

Makes sense. I will try to add this next week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants