The Inbox: Morphic-cbc.1665.mcz
Locked
 
|
A new version of Morphic was added to project The Inbox:
http://source.squeak.org/inbox/Morphic-cbc.1665.mcz ==================== Summary ==================== Name: Morphic-cbc.1665 Author: cbc Time: 5 June 2020, 7:38:31.284262 pm UUID: c887ffd7-40ae-3547-8240-13186db081c8 Ancestors: Morphic-cbc.1664 Add support for locking a morph so that it doesn't accept keyboard actions - but is still accessible with the mouse. Sending #rejectKeyboardFocus will make that morph completely ignore keyboard events of any kind. A morph can be tested with #shouldRejectKeyboardFocus to see if this should happen. DialogWindow has been modified to reject keyboard focus, so that in (say, ConfirmDialogs) you can still use the keyboard to interact with the keys AND you can interact with the message text via the mouse. (Also updated StringMorph similarly.) =============== Diff against Morphic-cbc.1664 =============== Item was changed: ----- Method: DialogWindow>>createMessage: (in category 'initialization') ----- createMessage: aString messageMorph := aString asText asMorph. messageMorph name: 'Message'; readOnly: true; setProperty: #indicateKeyboardFocus toValue: #never; + rejectKeyboardFocus. - lock. self setMessageParameters. ^ messageMorph! Item was added: + ----- Method: Morph>>rejectKeyboardFocus (in category 'accessing') ----- + rejectKeyboardFocus + self rejectKeyboardFocus: true.! Item was added: + ----- Method: Morph>>rejectKeyboardFocus: (in category 'accessing') ----- + rejectKeyboardFocus: boolean + self setProperty: #rejectKeyboardFocus toValue: boolean! Item was changed: ----- Method: Morph>>rejectsEvent: (in category 'events-processing') ----- rejectsEvent: anEvent "Return true to reject the given event. Rejecting an event means neither the receiver nor any of it's submorphs will be given any chance to handle it." + ^self isLocked or: [self visible not or: [anEvent isKeyboard and: [self shouldRejectKeyboardFocus]]]! - ^self isLocked or:[self visible not]! Item was added: + ----- Method: Morph>>rejectsKeyboardFocus (in category 'testing') ----- + rejectsKeyboardFocus + ^self valueOfProperty: #rejectKeyboardFocus ifAbsent: [false]! Item was added: + ----- Method: Morph>>shouldRejectKeyboardFocus (in category 'testing') ----- + shouldRejectKeyboardFocus + ^self valueOfProperty: #rejectKeyboardFocus ifAbsent: [false]! Item was changed: ----- Method: StringMorph>>wouldAcceptKeyboardFocus (in category 'event handling') ----- wouldAcceptKeyboardFocus + ^ self isLocked not and: [self shouldRejectKeyboardFocus not]! - ^ self isLocked not! |
|
|
This fixes the mangling I accidentally did to Marcel's changes, following his suggestion (roughly). I now get the best of both worlds in my image. If noone objects, I'll move it to trunk in a few days. -cbc On Fri, Jun 5, 2020 at 7:38 PM <[hidden email]> wrote: A new version of Morphic was added to project The Inbox: |
Re: The Inbox: Morphic-cbc.1665.mcz
|
|
Hi Chris,
what do you think about Morphic-ct.1666? :-) It would be nice if we did not completely lose the opportunity to manually activate the message and interact with it via keyboard.
Best, Christoph
Von: Squeak-dev <[hidden email]> im Auftrag von Chris Cunningham <[hidden email]>
Gesendet: Samstag, 6. Juni 2020 04:41:50 An: The general-purpose Squeak developers list Betreff: Re: [squeak-dev] The Inbox: Morphic-cbc.1665.mcz This fixes the mangling I accidentally did to Marcel's changes, following his suggestion (roughly).
I now get the best of both worlds in my image. If noone objects, I'll move it to trunk in a few days.
-cbc
On Fri, Jun 5, 2020 at 7:38 PM <[hidden email]> wrote:
A new version of Morphic was added to project The Inbox:
Carpe Squeak!
|
|
|
Ok. There are definite benefits with this. However, a stray mouse click still reduces the usefulness of a list chooser. Maybe ListChooser should be able to request the message rejects keyboard (this dialog is all about choosing a value, not about displaying a notice), while others could use ct.1666? At least with this the enter button will close the dialog - but allow full interaction with the message (as long as you start by using the mouse). -cbc On Sat, Jun 6, 2020 at 5:22 AM Thiede, Christoph <[hidden email]> wrote:
|
|
|
What do you think about Morphic-cbc.1667? It accepts your changes, removes mine, and still locks the message for (at present) 3 dialogs boxes in the image: - In ListChooser, when you have selected a partial of a choice, it asks to confirm a partial, a new, or cancel - In MonticelloConfirgurationBrowser, asking you to confirm saving the configuration - Any generic call to UserDialogBoxMorph>>confirm:trueChoice:falseChoice: (where it locks the static message text 'Please Confirm:') -cbc On Sat, Jun 6, 2020 at 9:36 AM Chris Cunningham <[hidden email]> wrote:
|
Re: The Inbox: Morphic-cbc.1665.mcz
|
|
Hi Chris,
I honestly don't think that we should individualize the behavior of the message body per kind of the dialog window. Even if there is a list chooser, it may still contain a relevant message text that I'd like to copy. And how often will such "stray mouse clicks" occur in real life? And if it occurred, you should have your fingers at your mouse anyway ;)
If at all, I would rather like to press Escape or Tab when the message text has the focus, in order to move the focus back to the dialog window. If we wanted this, we either could implement this in Editor or, via an event filter, in DialogWindow. What do you think?
Best, Christoph Von: Squeak-dev <[hidden email]> im Auftrag von Chris Cunningham <[hidden email]>
Gesendet: Samstag, 6. Juni 2020 19:13:57 An: The general-purpose Squeak developers list Betreff: Re: [squeak-dev] The Inbox: Morphic-cbc.1665.mcz What do you think about Morphic-cbc.1667?
It accepts your changes, removes mine, and still locks the message for (at present) 3 dialogs boxes in the image:
- In ListChooser, when you have selected a partial of a choice, it asks to confirm a partial, a new, or cancel
- In MonticelloConfirgurationBrowser, asking you to confirm saving the configuration
- Any generic call to UserDialogBoxMorph>>confirm:trueChoice:falseChoice: (where it locks the static message text 'Please Confirm:')
-cbc
On Sat, Jun 6, 2020 at 9:36 AM Chris Cunningham <[hidden email]> wrote:
Carpe Squeak!
|
|
|
Ok. I've been running with Morphic.ct.1666 for a few days now, and it works just fine for me. Nice work. The only real issue if you want to pick up the dialog and move it, and expect the text part to be grabbable (like it was) - the keyboard is then once again embedded in the text. But I've only done that once, and it is a weird thing to do for the confirm dialogs I'm talking about, so really unlikely. I'll probably reparent (and save) your package off of Morphic.mt.1666 just so that it looks nice and works well, likely tomorrow. Thanks -cbc On Sat, Jun 6, 2020 at 10:23 AM Thiede, Christoph <[hidden email]> wrote:
|
Re: The Inbox: Morphic-cbc.1665.mcz
|
|
Thank you for giving it a try! :-)
> The only real issue if you want to pick up the dialog and move it, and expect the text part to be grabbable (like it was) - the keyboard is then once again embedded in the text.
By the way, personally I use the blue mouse button at the most time I am moving anything in Morphic. It's a quite convenient way because no morph can regularly grab this event.
Best, Christoph Von: Squeak-dev <[hidden email]> im Auftrag von Chris Cunningham <[hidden email]>
Gesendet: Freitag, 12. Juni 2020 01:37:54 An: The general-purpose Squeak developers list Betreff: Re: [squeak-dev] The Inbox: Morphic-cbc.1665.mcz Ok. I've been running with Morphic.ct.1666 for a few days now, and it works just fine for me. Nice work.
The only real issue if you want to pick up the dialog and move it, and expect the text part to be grabbable (like it was) - the keyboard is then once again embedded in the text. But I've only done that once, and it is a weird thing to do for the confirm
dialogs I'm talking about, so really unlikely.
I'll probably reparent (and save) your package off of Morphic.mt.1666 just so that it looks nice and works well, likely tomorrow.
Thanks
-cbc
On Sat, Jun 6, 2020 at 10:23 AM Thiede, Christoph <[hidden email]> wrote:
Carpe Squeak!
|
«
Return to Squeak - Dev
|
1 view|%1 views
| Free forum by Nabble | Edit this page |