controlp5 | A gui library for processing.org

Laancelot's idea isn't bad, however, due to controlP5's OOP architecture it might not be possible, or at least trivial, to extend the controlP5's Button class to achieve your goal from a controlP5 sketch.

I'm making the assumption this is connected to other LED ring related questions you posted around this time (such as this one).

It might be wrong (and I'd be interested in seeing a solution), but without forking/modifying the controlP5 library itself, the inheritance chain gets a bit tricky.

Ideally you'd just need to extend Button and be done with it, however the method used to tell states (if a button over/pressed/etc.) is the Controller superclass's inside() method. To work with your image you'd need override this method to and swap the logic so it takes into account the alpha transparency of your png button skin (e.g. if the pixel under the cursor is opaque then it's inside otherwise it's not)

Here's a rough illustration you were constrained to using a Processing sketch:

As pointed out by @Andy the create_chart method is not defined within the Dane class in the above code example. This should be fixed first.

If you didn’t intend to add create_chart to a separate class, you might want to change

The problem list in:

Instead of setImage you can use setImages to add separate images for normal, hover, and down states.

Take a look at the example file for ControlP5button. You can open it in Processing by going to File > Examples... and looking for it in the ControlP5 folder.

In this part they load the 3 images for the button states for the play button:

I would recommend fixing syntax errors before posting and also removing any code that may not be related to the issue. Trying to cleanup / minimise code that way made the solution obvious for me so many times in the past.

I'm not sure you can group groups within groups. I might be wrong and hopefully someone more experienced with cp5 can correct me if I'm wrong.

If you use a single group and fix the syntax errors you could run something like this:

Reliable serial communication is not trivial. Ideally you would make you own binary communication protocol setting up a packet of bytes with a header that describes how many following bytes actually have data and perhaps even a checksum.

Strings will do to get started and it's great you're using nf() to make data easier to parse.

One potential gotcha could be the switch between putting a string together (rgbInput()) and reading one char at a time (char val = Serial.read();).

I would recommend breaking the problem down into smaller simpler parts, testing/debugging each part, then putting the parts back together one at a time to avoid integration bugs.

For example, the main challenge seems to be serial communication so I'd write a Processing and Arduino sketch to ensure that works reliably before adding in LED control.

Let's go for the option of \n terminated strings, even though it's sending a redundant extra character(e.g. port.write("2\n");) it would make buffering simpler:

• always buffer until a newline
• trim() the string for newline
• if the trimmed string has length 1 it's a command (like '2', 'l', 't'), otherwise it's a colour triplet

Here's a basic Arduino sketch that uses the ideas above (and a bit of your code ;) ):

Error:
It looks like you are nesting the button click handler functions. These are the actions that get triggered when your buttons are clicked. They don't control the layout or nesting of the buttons on screen.

Also, nested functions are not allowed in Processing, which is why you're getting the error.

Make sure the braces for your CONFIGURATION function don't contain any other functions:

You have a list, but you never add anything to that list. The list remains empty.

Drop the member field rect, delete this line:

It is in fact working (the rect.draw() function fires when you click the button). You can verify this by placing a print statement in the callback function: