<div><div dir="auto">Ok, the one line accessors seemed weird to me but I don’t mind. Then main thing is that the changes have CRs and shouldn’t, I guess they were added sometime after fileOut and before sending the email (for example if you edited the changes in a text editor, or perhaps it was the mail client) but it is easy to filter them out before doing a fileIn.</div></div><div dir="auto"><br></div><div dir="auto">I remember cmd-l, I don’t know why it disappeared, it was useful.</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 28 May 2020 at 12:48 PM, Casey Ransberger <<a href="mailto:bahweep@icloud.com">bahweep@icloud.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)"><div style="word-wrap:break-word;line-break:after-white-space">Wait, okay. Are newlines not displaying correctly, or are you just noticing that my lazy hands sometimes type an accessor method as<div><br></div><div>methodName ^ foo</div><div><br></div><div>?</div><div><br></div><div>Also, in Smalltalk, I usually put a trailing line at the end of a method because some folks might prefer mouse focus and accidentally write in. Having it on its own line makes it easier to stop panicking when *I* do that anyway! Incidentally, I remember being able to do Cmd-L to abandon my changes in a browser. Did that go away on purpose, or am I misremembering the keyboard shortcut?</div><div><br></div><div>I’m pretty sure the last time I was in on a discussion about coding style, there was just me and Juan and Juan said “you know what I want to see.” Which I took to mean, legible. If folks don’t like my one-line methods, I can totally take up a few more bytes with extra whitespace. And then make it up by leaving trailing whitespace off, if that’s what people want. I’m amenable. </div><div><br></div><div>If — on the other hand — my changes have affected the way methods I didn’t touch are displayed, yeah, that’s a huge problem, and a curious one at that. Not sure where that’d be coming from.</div><div><br></div><div>My confidence was at its lowest modifying CodeWindow>>decorateForInheritance. I wasn’t sure why it's presented so differently from every other sender of #useUniformColors, or why it had hard-coded colors in it. Against my nature, I left the hard-coded stuff alone and just got rid of the unneeded branch at "Theme current useUniformColors”.</div><div><br></div><div>Someone who knows CodeWindow better than I do might be able to provide insight, if there’s a problem here.</div><div><br></div><div>The only other thing I felt uncertain about was the change I made to SystemWindow>>windowColor. It originally did “^self model class windowColor”, but when I looked into that, it seemed that ultimately it was doing "Theme current object” anyway, so I changed it to that, as self model… blah blah seemed like the longer way around to find the value it was looking for, and I thought it odd that a model should care what color the bike shed is. Models and their visual representation generally being kept as separate as possible.</div><div><br></div><div>Anyway let me know if I’ve actually broken something… things seem to work fine for me, so I’ll need help to figure it out otherwise.</div></div><div style="word-wrap:break-word;line-break:after-white-space"><div><br></div><div>—Casey<br><div><br><blockquote type="cite"><div>On May 27, 2020, at 9:52 PM, Luciano Notarfrancesco <<a href="mailto:luchiano@gmail.com" target="_blank">luchiano@gmail.com</a>> wrote:</div><br><div><div dir="ltr"><div>I tried it and fileIns fine and everything works fine (including my Theme, I just should remove #useUniformColors from my Theme).</div><div>But there's a problem with newLines... some methods don't have newLines (for example UniformTheme>>browser looks like "browser ^ defaultColor") and most other methods have extra CRs.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, May 28, 2020 at 7:35 AM Casey Ransberger via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st" target="_blank">cuis-dev@lists.cuis.st</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">Found time to do the refactor today. Decoupling the Taskbar will have to wait a bit longer, as I’m out of time for now, but I expect that to be straightforward. The attached code is ready for review.<br>
<br>
Note for the integrator: this stuff touches a bunch of UI. Nothing blows up on my machine har har har! But best to back up your work before bringing in this code, as if I have a bug somewhere, you know, images might explode (usually a million debuggers will open if that’s the case.)<br>
<br>
I think on balance that this is a lot nicer than what was there before. Let me know what y’all think. As planned, it removes a bunch of code duplication and send sites, and adds a class with a small selection of entirely trivial methods. These changes shouldn’t affect observed behavior outside of perhaps an insignificant reduction in latency when opening new windows.<br>
<br>
—Casey<br>
<br>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
</blockquote></div>
</div></blockquote></div><br></div></div></blockquote></div></div>