<div dir="ltr">Thanks !<div><br></div><div>I'm rather new at this but I will try. News apace.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 17, 2023 at 12:48 PM Gerald Klix <<a href="mailto:cuis.01@klix.ch">cuis.01@klix.ch</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Alexandre,<br>
<br>
Subclassing BrowserWindow and PluggableButtonMorph is one possible solution.<br>
<br>
The other solution is to provide a proper change set for the base image,<br>
containing the added preference and the changed PluggableButtonMorph<br>
code. I presume that this is a rather simple change, meaning it does not<br>
create a big review effort for Juan, therefore he might accept such a <br>
change.<br>
I am strongly in favor of such a solution, because I want a better contrast<br>
of the buttons wrt. to the window background.<br>
<br>
While you are at it, you may also try to unify the code that calculates<br>
the color of the browser's inheritance button and the code that calculates<br>
the color of this very button's balloon help. The balloon help code<br>
is a rather bad hack (of mine).<br>
<br>
<br>
HTH and Best Regards,<br>
<br>
Gerald<br>
<br>
<br>
On 3/17/23 2:23 PM, Alexandre Rousseau via Cuis-dev wrote:<br>
> Hi.<br>
><br>
> I'm subclassing ClassicTheme to define my own theme, drawing examples from<br>
> ClassicTheme and its parent class (Theme).<br>
><br>
> I now want to customize the colors of Browser buttons (instances of<br>
> PluggableButtonMorph), i.e., I do not want their background color to be<br>
> taken from the defaultWindowColor theme variable, but rather from a new<br>
> one, say, defaultButtonBackgroundColor.<br>
><br>
> Does this imply that I have to subclass Browser and PluggableButtonMorph,<br>
> or is there a more expedient solution?<br>
><br>
> Thanks!<br>
><br>
> [image: Screenshot 2023-03-17 at 9.16.26 AM.png]<br>
><br>
><br>
<br>
</blockquote></div>