<div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000">
                                        
                                        
                                            
                                        
                                        
                                        Hi Martin.<div><br></div><div>> <span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">The check (aFont isNil) seems to imply that nil is a valid value here. But if</span></div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">you try, the method fails at [aFont emphasis].</span><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">Well, either aFont or emphasisCode can be nil. Not both. ;-) If I recall correctly, that's the existing usage I found when refactoring StringMorph last year in summer. It would be strange to support both arguments to be nil.</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">Background: Actually, the emphasis is part of the font object. However, it is possible to just store the emphasis in StringMorph and then fall back to the default font when needed -- with that emphasis. So, it is a bit of a mess, when you think about it. The user's model might separate font emphasis and font face. But the low-level implementation does not. Only StringMorph does.</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">Best,</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">Marcel</span></div><div class="mb_sig"></div>
                                        
                                        <blockquote class="history_container" type="cite" style="border-left-style: solid;border-width: 1px;margin-top: 20px;margin-left: 0px;padding-left: 10px;min-width: 500px">
                        <p style="color: #AAAAAA; margin-top: 10px;">Am 24.05.2020 15:26:35 schrieb Martin Kuball <martinkuball@web.de>:</p><div style="font-family:Arial,Helvetica,sans-serif">Hi!<br><br>I haven't used squeak for programing in years (yes, I know it's sad). Now that<br>I do, I noticed that there is no whisker package for 5.3. Well actually there<br>seems to be no package at all any more. So my question is, does anybody has a<br>whisker version that plays well with the current squeak version?<br><br>Meanwhile I loaded the version I used in 3.9. Had to fix some deprecated<br>warnings and some errors during initialization. I have it running now. But<br>it's missing some of the new features - like the icons - of the system<br>browser.<br><br>While fixing the errors I noticed the following method:<br><br>StringMorph>>font: aFont emphasis: emphasisCode<br><br>     self<br>          setFont: ((aFont isNil or: [aFont emphasis = emphasisCode] or:<br>[emphasisCode isNil])<br>                 ifTrue: [aFont]<br>                       ifFalse: [aFont emphasized: emphasisCode])<br>            emphasis: (emphasisCode ifNil: [aFont emphasis]).<br><br>The check (aFont isNil) seems to imply that nil is a valid value here. But if<br>you try, the method fails at [aFont emphasis].<br><br>Martin<br><br><br><br><br></div></blockquote></div>