Still, this approach feels flawed ... What is the idea behind checking #yellowButtonActivity: from Morph>>#yellowButtonActivity: at all? Can we come up with any better solution than this?<br>
<br>
Best,<br>
Christoph<br>
<br>
<font color="#808080">---<br>
</font><font color="#808080"><i>Sent from </i></font><font color="#808080"><i><a href="https://github.com/hpi-swa-lab/squeak-inbox-talk"><u><font color="#808080">Squeak Inbox Talk</font></u></a></i></font><br>
<br>
On 2022-09-13T17:54:05+00:00, commits@source.squeak.org wrote:<br>
<br>
> Christoph Thiede uploaded a new version of Morphic to project The Trunk:<br>
> http://source.squeak.org/trunk/Morphic-ct.2043.mcz<br>
> <br>
> ==================== Summary ====================<br>
> <br>
> Name: Morphic-ct.2043<br>
> Author: ct<br>
> Time: 13 September 2022, 7:54:00.818738 pm<br>
> UUID: e022b4f2-ce6b-584a-b084-1ce60b8cefeb<br>
> Ancestors: Morphic-ct.2042<br>
> <br>
> Fixes yellow-button menu invocation for pluggable lists that are not inside a system window. For instance, pluggable lists with in a dialog could not bring up their own menu previously but their containing dialog window's yellow-button menu was shown instead. See also Morphic-ct.1789.<br>
> <br>
> =============== Diff against Morphic-ct.2042 ===============<br>
> <br>
> Item was changed:<br>
>   ----- Method: PluggableListMorph>>createListMorph (in category 'initialization') -----<br>
>   createListMorph<br>
>   <br>
>       ^ self listMorphClass new<br>
>           listSource: self;<br>
>           cellInset: self class listMargins;<br>
>           hResizing: #spaceFill;<br>
>           vResizing: #shrinkWrap;<br>
>           cellPositioning: #leftCenter;<br>
>           setProperty: #indicateKeyboardFocus toValue: #never;<br>
> +         wantsYellowButtonMenu: false;<br>
>           yourself.!<br>
> <br>
> Item was changed:<br>
>   ----- Method: PluggableListMorph>>initialize (in category 'initialization') -----<br>
>   initialize<br>
>       listMorph := self createListMorph.<br>
>       super initialize.<br>
>       self scroller<br>
>           layoutPolicy: TableLayout new;<br>
> +         wantsYellowButtonMenu: false;<br>
> +         addMorph: listMorph.<br>
> -         addMorph: listMorph.    <br>
>       self<br>
>           minimumWidth: (self font widthOf: $m) * 5 ;<br>
>           minimumHeight: self font height<br>
>       <br>
>       !<br>
> <br>
>