1. PEP8
2. Я так понимаю что ask_color, mouse_motion2, , mouse_motion используются только внутри класса - тогда эти методы должны быть приватными, а не публичными.
3. mouse_motion1, mouse_motion2, mouse_motion3 - эти очень плохие названия попробуй назвать их так, чтобы по названию было понятно что делает конкретный метод, если они делают несколько вещей - разбей метод на несколько.
4.
self.canvas.bind('<Button-1>', self.mouse_motion)
self.canvas.bind('<Button-3>', self.mouse_motion2)
self.canvas.pack()
self.coordinates.pack()
self.prnt_color.pack()
self.button_color.pack(padx=20, pady=20)
- я бы вынес это в одельную функцию, которою уже вызывает __init__, хотя это опционально.
