<div dir="ltr">Hi Lucia!<div> Thank you for this new tool!</div><div> I have a few comments/improvements on the code you sent I'd like you to do:</div><div><br></div><div>1) It is not common to open a window sending the message #new. Let's make it open with the message #openInWorld</div><div>2) #openMenu should be #openInWorld (keep in mind you are redefining a super class method)</div><div>3) The size of the window is not relative to the size of the widgets. For example, if you go to Preferences>Size of GUI elements and select 32, open the window and you will see the window is not big enough</div><div>4) Move all the code to Theme-Themes package</div><div>6) The methods in Palettes category have the same strings repeated in different methods. I would extract to avoid repetition.</div><div>7) I would rename the names of the vars color1, color2, etc to sunset, ocean, etc.</div><div>8) Instead of using class variables in PersonalizedTheme, please use class instance variables.</div><div><br></div><div>  Please, send it again with all these changes :-)</div><div><br></div><div>Cheers!</div><div>Hernan.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jul 20, 2023 at 8:04 PM Lucia Secin via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st">cuis-dev@lists.cuis.st</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"><div dir="ltr">Hello,<div>  <br>I made a Theme Customizer for Cuis. With it you can easily change the colors of the windows, browser, background, text and labels. You can choose to use one of the predetermined colors or enter a custom hex code to fully personalize your theme.<div><br></div><div>I would like to add some more features in the future, but for now this is the finished product. I hope you like it and can include it in Cuis, I send attached the change set files.</div><div><br></div><div>In order for the Theme Customizer to work you also need to install the Personalized Theme, to open the customizer window, just create a new instance of ThemeCustomizerWindow.</div><div><br></div></div><div><img src="cid:ii_lkbr9qyn2" alt="Screenshot from 2023-07-20 20-00-03.png" width="209" height="44" style="margin-right: 0px;"><br></div><div><br></div><div>Regards,</div><div>Lucia.</div></div>
-- <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><br clear="all"><div><br></div><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div style="font-size:small"><div dir="ltr"><div dir="ltr"><div style="font-size:12.8px"><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span style="font-size:small"><font size="2"><span style="font-weight:normal"><span style="font-weight:bold">Hernán Wilkinson</span><br>Agile Software Development, Teaching & Coaching</span></font></span></span></strong></span></div><div style="font-size:12.8px"><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span style="font-size:small"><font size="2"><span style="font-weight:normal">Phone: +54-011</span></font></span></span></strong></span><font face="tahoma, sans-serif" size="2">-4893-2057</font></div><div style="font-size:12.8px"><strong style="font-family:tahoma,sans-serif;font-size:xx-small"><span style="font-size:8pt"><span style="font-size:small"><font size="2"><span style="font-weight:normal">Twitter: @HernanWilkinson</span></font></span></span></strong></div><div style="font-size:12.8px"><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span style="font-size:small"><font size="2"><span style="font-weight:normal">site: <a href="http://www.10pines.com/" style="color:rgb(17,65,112)" target="_blank">http://www.10Pines.com</a></span></font></span></span></strong></span></div><div style="font-size:12.8px"><font face="tahoma, sans-serif"><span style="border-collapse:collapse">Address: Alem 896</span></font>, Floor 6, Buenos Aires, Argentina</div></div></div></div></div></div></div></div>