[squeak-dev] The Inbox: Collections-kfr.1012.mcz
marcel.taeumel at hpi.de
Mon Jun 20 11:27:31 UTC 2022
Hi Karl --
> The whole image stops when you are holding down the mouse button on a text link. So there is room for improvement
Exactly. There are more symptoms of this issue. Yet, it is not advisable to degrade the implementation quality of TextAction when the strange code actually sits in TextEditor/NewParagraph.
Am 20.06.2022 13:05:00 schrieb karl ramberg <karlramberg at gmail.com>:
On Mon, Jun 20, 2022 at 11:39 AM Marcel Taeumel <marcel.taeumel at hpi.de [mailto:marcel.taeumel at hpi.de]> wrote:
Hi Karl --
Take a look at the following places to better understand, why we must not use #addDeferredUIMessage: in #actOnClickFor:
TextMorph >> #mouseDown:
TextEditor >> #mouseDown:
NewParagraph >> #clickAt:for:controller:
The expected control flow in the #mouseDown: handler is synchronous. Hmm... I recall a similar issue with #mouseDown: vs. #mouseUp: when implementing other buttons.
Usually, the reaction should be in #mouseUp:. In TextAction, the effective reaction is currently in the #mouseDown: handler of TextMorph even if TextEditor waits for this MVC-style mouse-up event in a loop. Hmm...
The whole image stops when you are holding down the mouse button on a text link. So there is room for improvement
Am 17.06.2022 10:23:31 schrieb Taeumel, Marcel <marcel.taeumel at hpi.de [mailto:marcel.taeumel at hpi.de]>:
-1 bc. wrong place. See senders of actOnClickFor: and my other answer. Let's work on that (bigger) issue after the release.
From: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org [mailto:squeak-dev-bounces at lists.squeakfoundation.org]> on behalf of commits at source.squeak.org [mailto:commits at source.squeak.org] <commits at source.squeak.org [mailto:commits at source.squeak.org]>
Sent: Thursday, June 16, 2022 9:59:32 PM
To: squeak-dev at lists.squeakfoundation.org [mailto:squeak-dev at lists.squeakfoundation.org] <squeak-dev at lists.squeakfoundation.org [mailto:squeak-dev at lists.squeakfoundation.org]>
Subject: [squeak-dev] The Inbox: Collections-kfr.1012.mcz
A new version of Collections was added to project The Inbox:
==================== Summary ====================
Time: 16 June 2022, 9:59:32.00337 pm
TextLinks would sometimes open i the background
=============== Diff against Collections-lrnp.1011 ===============
Item was changed:
----- Method: TextLink>>actOnClickFor: (in category 'event handling') -----
"Add to the end of the list. 'aClass selector', 'aClass Comment', 'aClass Definition', 'aClass Hierarchy' are the formats allowed."
+ Project current addDeferredUIMessage: [aMessageSet addItem: classAndMethod].
- aMessageSet addItem: classAndMethod.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Squeak-dev