These are very bad.
The Stop button doesn't return the play header to the start of the pattern. So you have a pause button right beside a pause button. Why does the Stop button exist at all?
Why is the Metronome under a shift layer but the Tap tempo isn't? That is backwards. How often does someone toggle the metronome, as opposed to create a custom tempo? Ya, a thousand times to one. Tap tempo should be under the shift layer and Metronome the direct button press.
Why does the loop button not stop the pattern from looping when it's disabled?
Why do I have to hold shift to erase things, when holding shift and pushing that button is supposed to make things replace?
What is happening with these basic functions? How did you manage to mangle this so badly? Big Yikes!