Why not subclassing?<br><br>As a general answer I&#39;d say:<br><br>Subclassing is a very strong (and static) relationship, and in Pharo we have only single inheritance + traits, so inheritance is a single shot gun :).<br>
<br>Splitting the behavior from the button opens you a door in your design.� You can design button actions with shortcuts, or undo actions, and choose your own inheritance criteria suiting your needs.<br><br>Just that, it lets an open door for richer designs.<br>
<br>Now, the ammount of code for one or other solution is pretty the same too, you just add one class with the toggle behavior the first one inherits from Button and the other does not), and then the instantiation code should not differ too much.<br>
<br>Bye,<br>Guille<br><br><div class="gmail_quote">On Tue, Nov 29, 2011 at 12:52 PM, Benjamin <span dir="ltr">&lt;<a href="mailto:benjamin.vanryseghem.pharo@gmail.com">benjamin.vanryseghem.pharo@gmail.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div style="word-wrap:break-word">Here is a little example of what I meant :)<div><br></div><div></div></div><br><div style="word-wrap:break-word">
<div></div><div><br></div><div><br></div><div>Ben</div><div><br><div><div>On Nov 28, 2011, at 11:28 PM, Ted F.A. van Gaalen wrote:</div><br><blockquote type="cite">Hi Guille, Ben<div>I&#39;ve just found this very convinient to subclass it and give it</div>
<div>its own behaviour.. what&#39;s wrong with that? Seems right OOP to me..</div><div>(I am a descendant of my mother and father (wow multiple inheritance :o)</div>
<div>in essence not very much different from my parents, still,</div><div>i exhibit other behaviour e.g. working with computers..)</div><div>But, yes, an example of what might be a better solution here</div><div>is appreciated, thanks</div>

<div>Ted</div><div>Anyone answering my other questions? Thanks</div><div><br></div><div>(i only have 6 apples no, I&#39;ve just eaten one)</div><div><br><br><div class="gmail_quote">On Mon, Nov 28, 2011 at 11:19 PM, Guillermo Polito <span dir="ltr">&lt;<a href="mailto:guillermopolito@gmail.com" target="_blank">guillermopolito@gmail.com</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In two words, Ben means to use composition over inheritance :P.<br><br>Guille<div><div><br><br>
<div class="gmail_quote">On Mon, Nov 28, 2011 at 7:10 PM, Benjamin <span dir="ltr">&lt;<a href="mailto:benjamin.vanryseghem.pharo@gmail.com" target="_blank">benjamin.vanryseghem.pharo@gmail.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Here, you are not changing the mechanism of the button, you are connecting a button to a model (when i click, i turn on/off fullscreen).<div>


<br></div><div>It&#39;s not a specific button (like a dunno, a button with 3 different states or stgh else).</div><div><br></div><div>Your button is a classical one, but you want a specific model to plug your button with :)</div>


<div><br></div><div><br></div><div>If you want, I could do a little piece of code to show what i mean :)</div><div><br></div><div><br></div><div><br></div><div>Ben</div><div><div><div><br></div><div><br></div><div>
<br><div><div>On Nov 28, 2011, at 11:03 PM, Ted F.A. van Gaalen wrote:</div><br><blockquote type="cite">Could you be more elaborate on this, Ben?<div>I don&#39;t see it (yet)</div><div>thanks</div><div>Ted<br><br><div class="gmail_quote">


On Mon, Nov 28, 2011 at 10:58 PM, Benjamin <span dir="ltr">&lt;<a href="mailto:benjamin.vanryseghem.pharo@gmail.com" target="_blank">benjamin.vanryseghem.pharo@gmail.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Because it&#39;s not a specific button, it&#39;s a simple button used in a specific way<div>



<br></div><div><br></div><div>Ben</div><div><div><div><br><div><div>On Nov 28, 2011, at 10:44 PM, Ted F.A. van Gaalen wrote:</div><br><blockquote type="cite">Hi Ben<div><br></div><div>All the functionality needed is just</div>



<div>to toggle a Display state..</div><div><br></div><div>then why use model?</div><div><br></div><div>Ted</div><div><br><div class="gmail_quote">On Mon, Nov 28, 2011 at 10:32 PM, Benjamin <span dir="ltr">&lt;<a href="mailto:benjamin.vanryseghem.pharo@gmail.com" target="_blank">benjamin.vanryseghem.pharo@gmail.com</a>&gt;</span> wrote:<br>




<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Good idea :)<div><br></div><div>but imo, it shouldn&#39;t be a subclass of�SimpleButtonMorph.</div>




<div>I think a class representing the model is more appropriate.</div><div><br></div><div><br></div><div>Ben</div><div><div><div><br></div><div><br><div><div>On Nov 28, 2011, at 10:08 PM, Ted F.A. van Gaalen wrote:</div>




<br><blockquote type="cite">Hi there<div><br></div><div>Thanks for creating version 1.3 !!!!!!!</div><div><br></div><div>-loaded Seaside into it.</div><div>-did a file-in of my Seaside pages</div><div>Works. OK</div><div>




<br></div><div>Still missing:</div>
<div>What I really would like�</div><div>perhaps in the next release</div><div>a Full Screen Toggle Button</div><div>somewhere on the desktop..</div><div><br></div><div>or -like in the Chrome Internet Browser</div><div>could you assign it to the F11 key?</div>





<div>just an idea.</div><div><br></div><div>Don&#39;t understand why it is hidden�</div><div>in Settings browser&gt;&gt;appearance&gt;&gt;desktop</div><div><br></div><div>(IMHO it does not belong there at all</div><div>it&#39;s not a Setting but just a window state..)</div>





<div><br></div><div>For now, I&#39;ve solved this by creating this little helper,</div><div>which, due to my�laziness�to give it coordinates,</div><div>drops a button in the upper left corner.</div><div>Mais enfin, it works.�</div>





<div><br></div><div>The class:</div><div><br></div><div><div><div>SimpleButtonMorph subclass: #FullScreenFlipFlopMorph</div><div><span style="white-space:pre-wrap">	</span>instanceVariableNames: &#39;&#39;</div>
<div><span style="white-space:pre-wrap">	</span>classVariableNames: &#39;&#39;</div><div><span style="white-space:pre-wrap">	</span>poolDictionaries: &#39;&#39;</div><div><span style="white-space:pre-wrap">	</span>category: &#39;TGUtil-One&#39;</div>





</div></div><div><br></div><div>with just this two instance methods:</div><div><div><br></div><div>initialize</div><div><span style="white-space:pre-wrap">	</span>super initialize.</div><div><span style="white-space:pre-wrap">	</span>self</div>





<div><span style="white-space:pre-wrap">		</span>target: self;</div><div><span style="white-space:pre-wrap">		</span>label: &#39;Full Screen&#39;;</div><div><span style="white-space:pre-wrap">		</span>color: Color orange;</div>





<div><span style="white-space:pre-wrap">		</span>actionSelector: #toggle;</div><div><span style="white-space:pre-wrap">		</span>openInWorld<span style="white-space:pre-wrap">	</span>&quot;target is self is Smalltalk;&quot;</div>





</div><div><div>toggle</div><div><span style="white-space:pre-wrap">	</span>Display toggleFullScreen</div></div><div><br></div><div>In a workspace brought to life with evaluating</div><div><br></div><div>
fullScreenFlipFlop := FullScreenFlipFlopMorph new.</div><div><br></div><div><br></div><div>Another question:</div><div><br></div><div>As 1.3 uses a cog VM.�</div><div>Is the <a href="http://seasidehosting.st/" target="_blank">seasidehosting.st</a></div>





<div>still using �a non-cog VM?</div><div>This would imply a cog VM</div><div>based image would not run on�</div><div>the seasidehosting?</div><div><br></div><div><br></div><div>What happened to the user window styles?�</div>





<div>(I only see Default and DemoMode)</div><div>I liked the sort of Aqua themes (I have 7 apples, they are real and in the kitchen :o)</div><div><br></div><div><br></div><div>Kind Regards</div><div>TedvG</div><div><br></div>





<div><br></div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div>
</blockquote></div><br></div></div></div></div></blockquote></div><br></div>
</blockquote></div><br></div></div></div></div></blockquote></div><br></div>
</blockquote></div><br></div></div></div></div></blockquote></div><br>
</div></div></blockquote></div><br></div>
</blockquote></div><br></div></div><br></blockquote></div><br>