Node use group

[vmjcv]

• Refactoring _ready()
• Manipulation buttons modulate themselves by use group manipulation_button: In the past, get_children was used. This is a very fragile structure. It relies too much on the internal structure of the card. Using groups can avoid this.
• Delete _on_button_mouse_exited:When floating on the card, this will cause an abnormal display
• Initialize the transparency of the manipulation_button control to ensure that the control is transparent
• The change of modulate should only be in “a” instead of all changes, use each control instead of the parent panel to make changes

[db0]

Looks good, but I would like to ask that in the future you also try to include some GUT tests for new functionality. I.e. you could add a test that check all individual buttons have been modulated in/out, one that checks that manipulation_buttons_self is being populated etc.

I will do this later for this change, but it helps when I don't have to :)

[db0]

This change needed a bit of work to fit the standard I'm expecting. I've done the modifications so you could view them as an example: Please see https://github.com/db0/godot-card-gaming/commit/eefb945840efbd615a427a7235309e0a4460f398 for the sort of quality I expect. Specifically please try to provide the following in your pull requests

• Static Typing
• Unit Tests
• Consistent Syntax to the existing codebase
• And when possible, self-explanatory variable names

I know it's a lot more work, but it significantly improves quality, troubleshooting and readability.