<div dir="ltr">I think you may be correct.<div><br></div><div>What I suspect happened is that I had loaded the Theme SAR before this was put into trunk, and when I migrated up afterwards, somehow it didn't migrate nicely.</div><div>In any case, what is in Trunk is what I proposed (with slightly different formatting - but same code).</div><div><br></div><div>Sorry for the disruption.</div><div><br></div><div>-cbc</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 1, 2017 at 6:03 PM, David T. Lewis <span dir="ltr"><<a href="mailto:lewis@mail.msen.com" target="_blank">lewis@mail.msen.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I'm confused, the new version of MorphicToolBuilder>><wbr>buildPluggableTree:<br>
from the inbox looks the same as the method currently in my trunk image,<br>
and I am not able to reproduce the problem with explorers in dark theme.<br>
<br>
Is there a version mixup here? Or maybe someone already merged the fix?<br>
<br>
Dave<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Feb 01, 2017 at 02:09:10PM -0800, Chris Cunningham wrote:<br>
> So, tracking this down, it was a change in 2/5/2016 (?) from kfr, in method<br>
> MorphicToolBuilder>><wbr>buildPluggableTree:, where it sends:<br>
> widget color: Preferences defaultBackgroundColor.<br>
> This if after the theme-based setting of the color, so it overrides<br>
> whatever the theme wants it to be.<br>
><br>
> Fix in the inbox, in:<br>
>     ToolBuilder-Morphic-cbc.191<br>
> Basically, I just removed the override of the color, to let the theme color<br>
> be used instead.<br>
><br>
> (as an aside, I'm not sure why in a selected dark theme, the<br>
> defualtBackgroundColor is "veryVeryLightGray").<br>
><br>
> -cbc<br>
><br>
> On Tue, Jan 31, 2017 at 5:06 PM, Chris Cunningham <<a href="mailto:cunningham.cb@gmail.com">cunningham.cb@gmail.com</a>><br>
> wrote:<br>
><br>
> > Hmm.  This only happens when i open a new explorer.  If I open an<br>
> > explorer, then change normal squeak theme, then go back to Monokai(dark),<br>
> > the tree view is now dark (like I would expect).  But the next explorer<br>
> > will have the white background again.<br>
> ><br>
> > -cbc<br>
> ><br>
> > On Tue, Jan 31, 2017 at 9:52 AM, Chris Cunningham <<a href="mailto:cunningham.cb@gmail.com">cunningham.cb@gmail.com</a><br>
> > > wrote:<br>
> ><br>
> >> I don't know where this happened, but in a pre-etoys-updated 6.0Alpha<br>
> >> image (16548), things like inspectors are readable:<br>
> >> [image: Inline image 1]<br>
> >> while in a post-etoys updated 6.0Alpha image (16903), these inspectors<br>
> >> are nearly useless:<br>
> >> [image: Inline image 2]<br>
> >> It may be that I had tweaked the colors in the older image, or loaded a<br>
> >> different code where someone else fixed it. I'll look for where the change<br>
> >> came from, but if someone knows what to fix, I'd appreciate that.<br>
> >><br>
> >> -cbc<br>
> >><br>
> ><br>
> ><br>
<br>
<br>
<br>
<br>
><br>
<br>
<br>
</div></div></blockquote></div><br></div>