[Cuis-dev] Theme Customizer

[Hernán Wilkinson]

Hi Lucia!
 Thank you for this new tool!
 I have a few comments/improvements on the code you sent I'd like you to do:

1) It is not common to open a window sending the message #new. Let's make
it open with the message #openInWorld
2) #openMenu should be #openInWorld (keep in mind you are redefining a
super class method)
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
4) Move all the code to Theme-Themes package
6) The methods in Palettes category have the same strings repeated in
different methods. I would extract to avoid repetition.
7) I would rename the names of the vars color1, color2, etc to sunset,
ocean, etc.
8) Instead of using class variables in PersonalizedTheme, please use class
instance variables.

  Please, send it again with all these changes :-)

Cheers!
Hernan.

On Thu, Jul 20, 2023 at 8:04 PM Lucia Secin via Cuis-dev <
cuis-dev at lists.cuis.st> wrote:

> Hello,
>
> 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.
>
> 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.
>
> 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.
>
> [image: Screenshot from 2023-07-20 20-00-03.png]
>
> Regards,
> Lucia.
> --
> Cuis-dev mailing list
> Cuis-dev at lists.cuis.st
> https://lists.cuis.st/mailman/listinfo/cuis-dev
>


-- 

*Hernán WilkinsonAgile Software Development, Teaching & Coaching*
*Phone: +54-011*-4893-2057
*Twitter: @HernanWilkinson*
*site: http://www.10Pines.com <http://www.10pines.com/>*
Address: Alem 896, Floor 6, Buenos Aires, Argentina
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20230721/ab6ecfa5/attachment-0001.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Screenshot from 2023-07-20 20-00-03.png
Type: image/png
Size: 19189 bytes
Desc: not available
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20230721/ab6ecfa5/attachment-0001.png>