If you'd like (not required), would you mind posting the result and maybe a little bit of the wiring you did?
Again: not required, but it may be helpful for others 😊 

Either way; glad to hear it works, thanks again for the coffee and enjoy the holidays! 😉 

end good , all good

Let me know whenever you have questions ... ideally through the forum though, so others can learn from it as well 😉 

Merry Christmas, and a Happy New Year - hope your daughter is excited about the gift 😉  👍 

#define BRIGHTNESS 100

just one more question

can you add a brightness function to the entire sketch please ?

Is it a big job to do one with a potentiometer?

if it's to much work just a setting in the sketch

You have been a big help Hans . I'm not gonna tweak it anymore or ask questions , because i can't get all the new features in the sketch  at work without errors.

My daughter is gonna be happy , and I think dad is a little to picky . Maybe I just want stuff in the sketch that she will never use .

I will try to install it on a ESP8266 in the future .

Happy holidays and keep save

The ESP8266 is in essence an Arduino on steroids ... faster, smaller, more memory, etc.

Using the WiFi part is optional. If you prefer to use a button and an RF switch, like you did with your Arduino, then you do not need to use the WiFi feature.

If you do want to use the WiFi option, then yes one of the methods to control the lights (for example) would be through a web-interface (call it a "mini website").
You will however have to provide the "content" for this yourself.
This can be by sending text (see the ESP8266 example) we can send through the server as a response to a request from the client (computer, cellphone, tablet, etc).

Another, more complex, approach is by using a SPIFFS file system on the flash part of an ESP8266. This would allow you to store actual text files and such on the ESP8266.
I would not recommend this method until you're more familiar with the ins and outs of Arduino programming and especially the technical features of the ESP8266.
Here an article that describes this. Again: not for beginners. 😉 

this gives me a error

backup:193:3: error: 'lastPressed' was not declared in this scope
lastPressed = millis();
^~~~~~~~~~~
exit status 1
expected identifier before '(' token

My first guess: It looks like I may have made a typo in this line (which can be found in changeEffect and changeEffect2):

if (digitalRead (BUTTON2) == HIGH) && ( lastPressed + bounceDelay < millis() ) ) {
                                                    woopsie one ")" too many --^

which should be

if (digitalRead (BUTTON2) == HIGH) && ( lastPressed + bounceDelay < millis() ) {

Error messages can be quite confusing. When looking at this one it tells me initially that "lastPressed" was not defined. Since I assume you added this line at the beginning, this seem odd:

unsigned long lastPressed = 0;  // last time the button was pressed in milliseconds

Right after that the error indicates an error with the round bracket "(" and ")", so I looked at that line an noticed I had typed one ")" too many.

Unfortunately, the code you  posted is not the one with the error, since the error points to line 193, which looks different in the code you attached.

The buttons also give me an error

void changeEffect() {
if (digitalRead (BUTTON) == HIGH) && ( lastPressed + bounceDelay < millis() ) ) {
lastPressed = millis();
selectedEffect++;
EEPROM.put(0, selectedEffect);
asm volatile (" jmp 0");
}
lastPressed = millis();
}

this gives me a error

backup:193:3: error: 'lastPressed' was not declared in this scope
lastPressed = millis();
^~~~~~~~~~~
exit status 1
expected identifier before '(' token

Yep , case 0 now displays correct....... Thx Hans

the 2 nd buttons I need to check later today ....