<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
On 5/17/2024 10:39 PM, Alan Ferrise via Cuis-dev wrote:
<blockquote
cite="mid:CALOEJBn9s_sw8MqzHWRt+6QjX_VSOhzQ_dSR1uMt0XJmO9syJQ@mail.gmail.com"
type="cite">
<div dir="ltr">Hi, I noted a small bug when I was doing some
exercises in Cuis so I tried to fix it:
<div><img src="cid:part1.01030105.09070007@cuis.st"
alt="image.png" style="margin-right: 0px;" height="316"
width="372"><br>
</div>
<div>When the mouse is over a button and you scroll, it ends up
scrolling the text that sits behind the active window, which
in this case is the Terse Guide.</div>
<div><br>
</div>
<div>I started looking how to fix this and I found that
PluggableScrollPane is the class responsible for modeling
scrolling windows. I noted that it doesn't listen to mouseOver
events, so I did it and added an instance collaborator named
mouseIsOver, whose value is checked when the mouseScroll event
is called. This seems to have fixed the problem.</div>
<div>I then looked its sibling class, PluggableButtonMorph, that
listens to the mouseOver events in the same manner that I end
up implementing it in PluggableScrollPane, which make me
wonder if I had to put mouseIsOver in their superclass,
however, I ended up discarding the idea.</div>
<div>I include the ChangeSet with the changes.</div>
<div><br>
</div>
<div>Thanks in advance.</div>
<div><br>
</div>
</div>
</blockquote>
<br>
Hi Alan,<br>
<br>
While I agree on the problem, I don't really agree on the suggested
solution. The problem is in the event dispatching logic that decides
to which morph deliver the scroll events. This should be solved
without adding new ivars at all. If nobody beats me to it, I'll do
it in a few days.<br>
<br>
Thanks,<br>
<pre class="moz-signature" cols="72">--
Juan Vuletich
cuis.st
github.com/jvuletich
researchgate.net/profile/Juan-Vuletich
independent.academia.edu/JuanVuletich
patents.justia.com/inventor/juan-manuel-vuletich
linkedin.com/in/juan-vuletich-75611b3
twitter.com/JuanVuletich</pre>
</body>
</html>