X-Git-Url: https://git.cinelerra-gg.org/git/?p=goodguy%2Fcinelerra.git;a=blobdiff_plain;f=cinelerra-5.1%2Fcinelerra%2Fcolorpicker.C;h=7c0a38d0d946fa8364b5e0c84c25ed442d2110e2;hp=78f1f957fd2ef72b20b6c70859d9b3fde2a47491;hb=7ffa6954689f14a41a3bae7a0fdb3f2abeaa9679;hpb=04293346a5ef49683cfa6ca3a98ef6cbfcdf7732

diff --git a/cinelerra-5.1/cinelerra/colorpicker.C b/cinelerra-5.1/cinelerra/colorpicker.C
index 78f1f957..7c0a38d0 100644
--- a/cinelerra-5.1/cinelerra/colorpicker.C
+++ b/cinelerra-5.1/cinelerra/colorpicker.C
@@ -79,13 +79,14 @@ BC_Window* ColorPicker::new_gui()
 	int w = ColorWindow::calculate_w();
 	int h = ColorWindow::calculate_h();
 	if( ok_cancel )
-		h += bmax(BC_OKButton::calculate_h(),BC_CancelButton::calculate_h());
+		h += bmax(ColorOK::calculate_h(),ColorCancel::calculate_h());
 	int root_w = display_info.get_root_w(), root_h = display_info.get_root_h();
 	if( x+w > root_w ) x = root_w - w;
 	if( y+h > root_h ) y = root_h - h;
 	if( x < 0 ) x = 0;
 	if( y < 0 ) y = 0;
 	ColorWindow *window = new ColorWindow(this, x, y, w, h, window_title);
+	window->create_objects();
 	window->start_selection(color, !do_alpha ? -1 : alpha, ok_cancel);
 	return window;
 }
@@ -114,6 +115,13 @@ ColorWindow::~ColorWindow()
 {
 }
 
+void ColorWindow::create_objects()
+{
+	BC_WindowBase::create_objects();
+	ColorGUI::create_objects();
+	thread->create_objects(this);
+}
+
 void ColorWindow::update_gui(int color, int alpha)
 {
 	lock_window("ColorWindow::update_gui");
@@ -266,8 +274,8 @@ void ColorGUI::create_objects()
 		aph_a->create_objects();  aph_a->set_tooltip(_("Alpha"));
 	}
 	if( ok_cancel ) {
-		add_tool(new BC_OKButton(window));
-		add_tool(new BC_CancelButton(window));
+		add_tool(new ColorOK(this, window));
+		add_tool(new ColorCancel(this, window));
 	}
 	create_objects(this);
 
@@ -289,8 +297,41 @@ void ColorGUI::change_values()
 }
 
 
+ColorOK::ColorOK(ColorGUI *gui, BC_WindowBase *window)
+ : BC_OKButton(window)
+{
+	this->gui = gui;
+	this->window = window;
+}
+int ColorOK::handle_event()
+{
+	gui->ok_cancel = 0;
+	gui->close_gui();
+	window->sync_display();
+	return 1;
+}
+
+ColorCancel::ColorCancel(ColorGUI *gui, BC_WindowBase *window)
+ : BC_CancelButton(window)
+{
+	this->gui = gui;
+	this->window = window;
+}
+int ColorCancel::handle_event()
+{
+	gui->ok_cancel = 1;
+	gui->close_gui();
+	window->sync_display();
+	return 1;
+}
+
 int ColorGUI::close_gui()
 {
+	if( button_grabbed ) {
+		button_grabbed = 0;
+		window->ungrab_buttons();
+		window->ungrab_cursor();
+	}
 	window->set_done(ok_cancel ? 1 : 0);
 	return 1;
 }
@@ -1141,7 +1182,7 @@ int PaletteGrabButton::handle_event()
 	if( gui->window->grab_buttons() ) {
 		gui->window->grab_cursor();
 		gui->button_grabbed = 1;
-		gui->button_press_gui(); // redraw face HI
+		return BC_Button::button_press_event(); // redraw face HI
 	}
 	return 1;
 }