X-Git-Url: http://git.cinelerra-gg.org/git/?p=goodguy%2Fhistory.git;a=blobdiff_plain;f=cinelerra-5.1%2Fcinelerra%2Fcwindowgui.C;h=95bae3979ce8071ca20358da276f75e33ea4b67c;hp=172ee6e3cbbc88c115aa80fdb8887d4b964009da;hb=0513350234a8dcd08e5a0117d5121724ef7b76b6;hpb=2f28632d8209ef735de1698a596bc94fa9613497

diff --git a/cinelerra-5.1/cinelerra/cwindowgui.C b/cinelerra-5.1/cinelerra/cwindowgui.C
index 172ee6e3..95bae397 100644
--- a/cinelerra-5.1/cinelerra/cwindowgui.C
+++ b/cinelerra-5.1/cinelerra/cwindowgui.C
@@ -92,23 +92,29 @@ CWindowGUI::CWindowGUI(MWindow *mwindow, CWindow *cwindow)
 	this->mwindow = mwindow;
 	this->cwindow = cwindow;
 	affected_track = 0;
-	affected_x = 0;
-	affected_y = 0;
-	affected_z = 0;
+	affected_x = affected_y = affected_z = 0;
 	mask_keyframe = 0;
 	orig_mask_keyframe = new MaskAuto(0, 0);
 	affected_point = 0;
-	x_offset = 0;
-	y_offset = 0;
-	x_origin = 0;
-	y_origin = 0;
+	x_offset = y_offset = 0;
+	x_origin = y_origin = 0;
 	current_operation = CWINDOW_NONE;
 	tool_panel = 0;
-	translating_zoom = 0;
 	active = 0;
 	inactive = 0;
-	crop_translate = 0;
+	crop_handle = -1; crop_translate = 0;
+	crop_origin_x = crop_origin_y = 0;
+	crop_origin_x1 = crop_origin_y1 = 0;
+	crop_origin_x2 = crop_origin_y2 = 0;
 	eyedrop_visible = 0;
+	eyedrop_x = eyedrop_y = 0;
+	ruler_origin_x = ruler_origin_y = 0;
+	ruler_handle = -1; ruler_translate = 0;
+	center_x = center_y = center_z = 0;
+	control_in_x = control_in_y = 0;
+	control_out_x = control_out_y = 0;
+	translating_zoom = 0;
+	highlighted = 0;
 }
 
 CWindowGUI::~CWindowGUI()
@@ -169,10 +175,10 @@ void CWindowGUI::create_objects()
 	timebar->create_objects();
 
 #ifdef USE_SLIDER
- 	add_subwindow(slider = new CWindowSlider(mwindow, 
- 		cwindow, 
+ 	add_subwindow(slider = new CWindowSlider(mwindow,
+ 		cwindow,
 		mwindow->theme->cslider_x,
- 		mwindow->theme->cslider_y, 
+ 		mwindow->theme->cslider_y,
  		mwindow->theme->cslider_w));
 #endif
 
@@ -199,8 +205,9 @@ void CWindowGUI::create_objects()
 		mwindow->theme->czoom_y,
 		mwindow->theme->czoom_w);
 	zoom_panel->create_objects();
-	zoom_panel->zoom_text->add_item(new BC_MenuItem(AUTO_ZOOM));
-	if(!mwindow->edl->session->cwindow_scrollbars) zoom_panel->set_text(AUTO_ZOOM);
+	zoom_panel->zoom_text->add_item(new BC_MenuItem(auto_zoom = _(AUTO_ZOOM)));
+	if( !mwindow->edl->session->cwindow_scrollbars )
+		zoom_panel->set_text(auto_zoom);
 
 // 	destination = new CWindowDestination(mwindow,
 // 		this,
@@ -243,7 +250,7 @@ int CWindowGUI::resize_event(int w, int h)
 	flash(0);
 
 	composite_panel->reposition_buttons(mwindow->theme->ccomposite_x,
-		mwindow->theme->ccomposite_y);
+		mwindow->theme->ccomposite_y, mwindow->theme->ccomposite_h);
 
 	canvas->reposition_window(mwindow->edl,
 		mwindow->theme->ccanvas_x,
@@ -255,7 +262,7 @@ int CWindowGUI::resize_event(int w, int h)
 
 #ifdef USE_SLIDER
  	slider->reposition_window(mwindow->theme->cslider_x,
- 		mwindow->theme->cslider_y, 
+ 		mwindow->theme->cslider_y,
  		mwindow->theme->cslider_w);
 // Recalibrate pointer motion range
 	slider->set_position();
@@ -357,44 +364,38 @@ void CWindowGUI::draw_status(int flush)
 		flush);
 }
 
-
-void CWindowGUI::zoom_canvas(int do_auto, double value, int update_menu)
+float CWindowGUI::get_auto_zoom()
 {
-	if(do_auto)
-		mwindow->edl->session->cwindow_scrollbars = 0;
-	else
-		mwindow->edl->session->cwindow_scrollbars = 1;
+	float conformed_w, conformed_h;
+	mwindow->edl->calculate_conformed_dimensions(0, conformed_w, conformed_h);
+	float zoom_x = canvas->w / conformed_w;
+	float zoom_y = canvas->h / conformed_h;
+	return zoom_x < zoom_y ? zoom_x : zoom_y;
+}
 
-	float old_zoom = mwindow->edl->session->cwindow_zoom;
-	float new_zoom = value;
-	float x = canvas->w / 2.0;
-	float y = canvas->h / 2.0;
-	canvas->canvas_to_output(mwindow->edl,
-				0,
-				x,
-				y);
-	x -= canvas->w_visible / 2 * old_zoom / new_zoom;
-	y -= canvas->h_visible / 2 * old_zoom / new_zoom;
-	if(update_menu)
-	{
-		if(do_auto)
-		{
-			zoom_panel->update(AUTO_ZOOM);
-		}
-		else
-		{
-			zoom_panel->update(value);
-		}
+void CWindowGUI::zoom_canvas(double value, int update_menu)
+{
+	float x = 0, y = 0;
+	float zoom = !value ? get_auto_zoom() : value;
+	mwindow->edl->session->cwindow_scrollbars = !value ? 0 : 1;
+	if( value ) {
+		float cx = canvas->get_xscroll() + 0.5f*canvas->w_visible;
+		float cy = canvas->get_yscroll() + 0.5f*canvas->h_visible;
+		float output_x = cx, output_y = cy;
+		canvas->output_to_canvas(mwindow->edl, 0, cx, cy);
+		x = output_x - cx / zoom;
+		y = output_y - cy / zoom;
 	}
+	canvas->update_zoom((int)(x+0.5), (int)(y+0.5), zoom);
+
+	if( update_menu )
+		zoom_panel->update(value);
+	if( mwindow->edl->session->cwindow_operation == CWINDOW_ZOOM )
+		composite_panel->cpanel_zoom->update(zoom);
 
-	canvas->update_zoom((int)x,
-		(int)y,
-		new_zoom);
 	canvas->reposition_window(mwindow->edl,
-		mwindow->theme->ccanvas_x,
-		mwindow->theme->ccanvas_y,
-		mwindow->theme->ccanvas_w,
-		mwindow->theme->ccanvas_h);
+		mwindow->theme->ccanvas_x, mwindow->theme->ccanvas_y,
+		mwindow->theme->ccanvas_w, mwindow->theme->ccanvas_h);
 	canvas->draw_refresh();
 }
 
@@ -458,7 +459,7 @@ int CWindowGUI::keypress_event()
 			mwindow->gui->unlock_window();
 			lock_window("CWindowGUI::keypress_event 2");
 			break;
-		case BACKSPACE:
+		case DELETE:
 			unlock_window();
 			mwindow->gui->lock_window("CWindowGUI::keypress_event 2");
 			mwindow->clear_entry();
@@ -569,43 +570,26 @@ void CWindowGUI::drag_motion()
 {
 	if(get_hidden()) return;
 
-	if(mwindow->session->current_operation == DRAG_ASSET ||
-		mwindow->session->current_operation == DRAG_VTRANSITION ||
-		mwindow->session->current_operation == DRAG_VEFFECT)
-	{
-		int old_status = mwindow->session->ccanvas_highlighted;
-		int cursor_x = get_relative_cursor_x();
-		int cursor_y = get_relative_cursor_y();
-
-		mwindow->session->ccanvas_highlighted = get_cursor_over_window() &&
-			cursor_x >= canvas->x &&
-			cursor_x < canvas->x + canvas->w &&
-			cursor_y >= canvas->y &&
-			cursor_y < canvas->y + canvas->h;
-
-
-		if(old_status != mwindow->session->ccanvas_highlighted)
-			canvas->draw_refresh();
-	}
+	if(mwindow->session->current_operation != DRAG_ASSET &&
+		mwindow->session->current_operation != DRAG_VTRANSITION &&
+		mwindow->session->current_operation != DRAG_VEFFECT) return;
+	int need_highlight = cursor_above() && get_cursor_over_window();
+	if( highlighted == need_highlight ) return;
+	highlighted = need_highlight;
+	canvas->draw_refresh();
 }
 
 int CWindowGUI::drag_stop()
 {
 	int result = 0;
 	if(get_hidden()) return 0;
-
-	if((mwindow->session->current_operation == DRAG_ASSET ||
-		mwindow->session->current_operation == DRAG_VTRANSITION ||
-		mwindow->session->current_operation == DRAG_VEFFECT) &&
-		mwindow->session->ccanvas_highlighted)
-	{
-// Hide highlighting
-		mwindow->session->ccanvas_highlighted = 0;
-		canvas->draw_refresh();
-		result = 1;
-	}
-	else
-		return 0;
+	if( !highlighted ) return 0;
+	if( mwindow->session->current_operation != DRAG_ASSET &&
+	    mwindow->session->current_operation != DRAG_VTRANSITION &&
+	    mwindow->session->current_operation != DRAG_VEFFECT) return 0;
+	highlighted = 0;
+	canvas->draw_refresh();
+	result = 1;
 
 	if(mwindow->session->current_operation == DRAG_ASSET)
 	{
@@ -620,7 +604,7 @@ int CWindowGUI::drag_stop()
 		if(mwindow->session->drag_assets->total)
 		{
 			mwindow->gui->lock_window("CWindowGUI::drag_stop 1");
-			mwindow->clear(0);
+			mwindow->clear(0, 1);
 			mwindow->load_assets(mwindow->session->drag_assets,
 				mwindow->edl->local_session->get_selectionstart(),
 				LOADMODE_PASTE,
@@ -635,7 +619,7 @@ int CWindowGUI::drag_stop()
 		if(mwindow->session->drag_clips->total)
 		{
 			mwindow->gui->lock_window("CWindowGUI::drag_stop 2");
-			mwindow->clear(0);
+			mwindow->clear(0, 1);
 			mwindow->paste_edls(mwindow->session->drag_clips,
 				LOADMODE_PASTE,
 				mwindow->session->track_highlighted,
@@ -734,13 +718,16 @@ CWrapper(set_inpoint)
 CWrapper(set_outpoint)
 CWrapper(clear_inpoint)
 CWrapper(clear_outpoint)
-CWrapper(to_clip)
 CWrapper(toggle_label)
 CWrapper(prev_label)
 CWrapper(next_label)
 CWrapper(prev_edit)
 CWrapper(next_edit)
 
+void CWindowEditing::to_clip()
+{
+        mwindow->to_clip(mwindow->edl, _("composer window: "));
+}
 
 
 CWindowMeters::CWindowMeters(MWindow *mwindow,
@@ -778,15 +765,8 @@ int CWindowMeters::change_status_event(int new_status)
 
 
 CWindowZoom::CWindowZoom(MWindow *mwindow, CWindowGUI *gui, int x, int y, int w)
- : ZoomPanel(mwindow,
- 	gui,
-	(double)mwindow->edl->session->cwindow_zoom,
-	x,
-	y,
-	w,
-	my_zoom_table,
-	total_zooms,
-	ZOOM_PERCENTAGE)
+ : ZoomPanel(mwindow, gui, (double)mwindow->edl->session->cwindow_zoom,
+	x, y, w, my_zoom_table, total_zooms, ZOOM_PERCENTAGE)
 {
 	this->mwindow = mwindow;
 	this->gui = gui;
@@ -796,17 +776,26 @@ CWindowZoom::~CWindowZoom()
 {
 }
 
-int CWindowZoom::handle_event()
+void CWindowZoom::update(double value)
 {
-	if(!strcasecmp(AUTO_ZOOM, get_text()))
-	{
-		gui->zoom_canvas(1, get_value(), 0);
+	char string[BCSTRLEN];
+	const char *cp = string;
+	if( value ) {
+		this->value = value;
+		int frac = value >= 1.0f ? 1 :
+			   value >= 0.1f ? 2 :
+			   value >= .01f ? 3 : 4;
+		sprintf(string, "x %.*f", frac, value);
 	}
 	else
-	{
-		gui->zoom_canvas(0, get_value(), 0);
-	}
+		cp = gui->auto_zoom;
+	ZoomPanel::update(cp);
+}
 
+int CWindowZoom::handle_event()
+{
+	double value = !strcasecmp(gui->auto_zoom, get_text()) ? 0 : get_value();
+	gui->zoom_canvas(value, 0);
 	return 1;
 }
 
@@ -814,13 +803,13 @@ int CWindowZoom::handle_event()
 
 #ifdef USE_SLIDER
 CWindowSlider::CWindowSlider(MWindow *mwindow, CWindow *cwindow, int x, int y, int pixels)
- : BC_PercentageSlider(x, 
+ : BC_PercentageSlider(x,
 			y,
 			0,
-			pixels, 
-			pixels, 
-			0, 
-			1, 
+			pixels,
+			pixels,
+			0,
+			1,
 			0)
 {
 	this->mwindow = mwindow;
@@ -840,20 +829,20 @@ int CWindowSlider::handle_event()
 
 void CWindowSlider::set_position()
 {
-	double new_length = mwindow->edl->tracks->total_playable_length();
+	double new_length = mwindow->edl->tracks->total_length();
 // 	if(mwindow->edl->local_session->preview_end <= 0 ||
 // 		mwindow->edl->local_session->preview_end > new_length)
 // 		mwindow->edl->local_session->preview_end = new_length;
-// 	if(mwindow->edl->local_session->preview_start > 
+// 	if(mwindow->edl->local_session->preview_start >
 // 		mwindow->edl->local_session->preview_end)
 // 		mwindow->edl->local_session->preview_start = 0;
 
 
-	update(mwindow->theme->cslider_w, 
-		mwindow->edl->local_session->get_selectionstart(1), 
+	update(mwindow->theme->cslider_w,
+		mwindow->edl->local_session->get_selectionstart(1),
 		0,
 		new_length);
-//		mwindow->edl->local_session->preview_start, 
+//		mwindow->edl->local_session->preview_start,
 //		mwindow->edl->local_session->preview_end);
 }
 
@@ -876,22 +865,22 @@ int CWindowSlider::decrease_value()
 
 
 // CWindowDestination::CWindowDestination(MWindow *mwindow, CWindowGUI *cwindow, int x, int y)
-//  : BC_PopupTextBox(cwindow, 
-//  	&cwindow->destinations, 
+//  : BC_PopupTextBox(cwindow,
+//  	&cwindow->destinations,
 // 	cwindow->destinations.values[cwindow->cwindow->destination]->get_text(),
-// 	x, 
-// 	y, 
-// 	70, 
+// 	x,
+// 	y,
+// 	70,
 // 	200)
 // {
 // 	this->mwindow = mwindow;
 // 	this->cwindow = cwindow;
 // }
-// 
+//
 // CWindowDestination::~CWindowDestination()
 // {
 // }
-// 
+//
 // int CWindowDestination::handle_event()
 // {
 // 	return 1;
@@ -988,7 +977,7 @@ void CWindowCanvas::update_zoom(int x, int y, float zoom)
 
 void CWindowCanvas::zoom_auto()
 {
-	gui->zoom_canvas(1, 1.0, 1);
+	gui->zoom_canvas(0, 1);
 }
 
 int CWindowCanvas::get_xscroll()
@@ -1584,6 +1573,7 @@ int CWindowCanvas::do_mask(int &redraw, int &rerender,
 					}
 					else {
 						if(!gui->shift_down()) {
+							output_to_canvas(mwindow->edl, 0, canvas_x0, canvas_y0);
 							if(test_bbox(cursor_x, cursor_y, canvas_x0, canvas_y0)) {
 								selected_point = i;
 							}
@@ -1605,6 +1595,7 @@ int CWindowCanvas::do_mask(int &redraw, int &rerender,
 					}
 					else if(i < points.size() - 1) {
 						if(!gui->shift_down()) {
+							output_to_canvas(mwindow->edl, 0, canvas_x3, canvas_y3);
 							if(test_bbox(cursor_x, cursor_y, canvas_x3, canvas_y3)) {
 								selected_point = (i < points.size() - 1 ? i + 1 : 0);
 							}
@@ -1641,15 +1632,14 @@ int CWindowCanvas::do_mask(int &redraw, int &rerender,
 // sprintf(string, "%d", (i < points.size() - 1 ? i + 1 : 0));
 // canvas->draw_text((int)canvas_x + CONTROL_W, (int)canvas_y + CONTROL_W, string);
 							}
-
-// Draw second control point.  Discard x2 and y2 after this.
-							output_to_canvas(mwindow->edl, 0, x2, y2);
+// Draw second control point.
+							output_to_canvas(mwindow->edl, 0, canvas_x2, canvas_y2);
 							get_canvas()->draw_line(
 								(int)canvas_x, (int)canvas_y,
 								(int)canvas_x2, (int)canvas_y2);
 							get_canvas()->draw_rectangle(
-								(int)x2 - CONTROL_W / 2,
-								(int)y2 - CONTROL_H / 2,
+								(int)canvas_x2 - CONTROL_W / 2,
+								(int)canvas_y2 - CONTROL_H / 2,
 								CONTROL_W, CONTROL_H);
 						}
 					}
@@ -1664,7 +1654,7 @@ int CWindowCanvas::do_mask(int &redraw, int &rerender,
 							(int)canvas_x - FIRST_CONTROL_W,
 							(int)canvas_y - FIRST_CONTROL_H,
 							mask_label);
-							
+
 						get_canvas()->draw_disc(
 							(int)canvas_x - FIRST_CONTROL_W / 2,
 							(int)canvas_y - FIRST_CONTROL_H / 2,
@@ -2177,15 +2167,24 @@ int CWindowCanvas::do_eyedrop(int &rerender, int button_press, int draw)
 	float blue = (float)*row++ / max; \
 	if(do_yuv) \
 	{ \
-		mwindow->edl->local_session->red += red + V_TO_R * (blue - 0.5); \
-		mwindow->edl->local_session->green += red + U_TO_G * (green - 0.5) + V_TO_G * (blue - 0.5); \
-		mwindow->edl->local_session->blue += red + U_TO_B * (green - 0.5); \
+		float r = red + V_TO_R * (blue - 0.5); \
+		float g = red + U_TO_G * (green - 0.5) + V_TO_G * (blue - 0.5); \
+		float b = red + U_TO_B * (green - 0.5); \
+		mwindow->edl->local_session->red += r; \
+		mwindow->edl->local_session->green += g; \
+		mwindow->edl->local_session->blue += b; \
+		if(r > mwindow->edl->local_session->red_max) mwindow->edl->local_session->red_max = r; \
+		if(g > mwindow->edl->local_session->green_max) mwindow->edl->local_session->green_max = g; \
+		if(b > mwindow->edl->local_session->blue_max) mwindow->edl->local_session->blue_max = b; \
 	} \
 	else \
 	{ \
 		mwindow->edl->local_session->red += red; \
 		mwindow->edl->local_session->green += green; \
 		mwindow->edl->local_session->blue += blue; \
+		if(red > mwindow->edl->local_session->red_max) mwindow->edl->local_session->red_max = red; \
+		if(green > mwindow->edl->local_session->green_max) mwindow->edl->local_session->green_max = green; \
+		if(blue > mwindow->edl->local_session->blue_max) mwindow->edl->local_session->blue_max = blue; \
 	} \
 }
 
@@ -2194,6 +2193,9 @@ int CWindowCanvas::do_eyedrop(int &rerender, int button_press, int draw)
 			mwindow->edl->local_session->red = 0;
 			mwindow->edl->local_session->green = 0;
 			mwindow->edl->local_session->blue = 0;
+			mwindow->edl->local_session->red_max = 0;
+			mwindow->edl->local_session->green_max = 0;
+			mwindow->edl->local_session->blue_max = 0;
 			for(int i = row1; i < row2; i++)
 			{
 				for(int j = column1; j < column2; j++)
@@ -2285,7 +2287,7 @@ void CWindowCanvas::draw_overlays()
 		get_canvas()->set_opaque();
 	}
 
-	if(mwindow->session->ccanvas_highlighted)
+	if(gui->highlighted)
 	{
 		get_canvas()->set_color(WHITE);
 		get_canvas()->set_inverse();
@@ -2765,8 +2767,8 @@ void CWindowCanvas::draw_bezier(int do_camera)
 
 }
 
-int CWindowCanvas::test_bezier(int button_press, 
-	int &redraw, 
+int CWindowCanvas::test_bezier(int button_press,
+	int &redraw,
 	int &redraw_canvas,
 	int &rerender,
 	int do_camera)
@@ -2831,11 +2833,11 @@ int CWindowCanvas::test_bezier(int button_press,
 					FloatAuto *previous = 0;
 					FloatAuto *next = 0;
 					float new_z = affected_z_autos->get_value(
-						track_position, 
+						track_position,
 						PLAY_FORWARD,
 						previous,
 						next);
-					gui->affected_z = 
+					gui->affected_z =
 						(FloatAuto*)gui->cwindow->calculate_affected_auto(
 							affected_z_autos, 1, &created, 0);
 					if(created) {
@@ -2848,25 +2850,25 @@ int CWindowCanvas::test_bezier(int button_press,
 					FloatAuto *previous = 0;
 					FloatAuto *next = 0;
 					float new_x = affected_x_autos->get_value(
-						track_position, 
+						track_position,
 						PLAY_FORWARD,
 						previous,
 						next);
 					previous = 0;
 					next = 0;
 					float new_y = affected_y_autos->get_value(
-						track_position, 
+						track_position,
 						PLAY_FORWARD,
 						previous,
 						next);
-					gui->affected_x = 
+					gui->affected_x =
 						(FloatAuto*)gui->cwindow->calculate_affected_auto(
 							affected_x_autos, 1, &created, 0);
 					if(created) {
 						gui->affected_x->set_value(new_x);
 						redraw_canvas = 1;
 					}
-					gui->affected_y = 
+					gui->affected_y =
 						(FloatAuto*)gui->cwindow->calculate_affected_auto(
 							affected_y_autos, 1, &created, 0);
 					if(created) {
@@ -2941,13 +2943,13 @@ int CWindowCanvas::test_bezier(int button_press,
 			else
 				mwindow->undo->update_undo_before(_("projector"), this);
 
-			gui->current_operation = 
+			gui->current_operation =
 				mwindow->edl->session->cwindow_operation;
 			gui->tool_panel->raise_window();
 			result = 1;
 		}
 	}
-	
+
 	return result;
 }
 
@@ -2955,84 +2957,50 @@ int CWindowCanvas::test_bezier(int button_press,
 int CWindowCanvas::test_zoom(int &redraw)
 {
 	int result = 0;
-	float zoom = get_zoom();
-	float x;
-	float y;
-
-	if(!mwindow->edl->session->cwindow_scrollbars)
-	{
-		mwindow->edl->session->cwindow_scrollbars = 1;
-		zoom = 1.0;
-		x = mwindow->edl->session->output_w / 2;
-		y = mwindow->edl->session->output_h / 2;
-		x = x - w / zoom / 2;
-		y = y - h / zoom / 2;
-	}
-	else
-	{
-// Cursor position relative to canvas
-		x = get_cursor_x();
-		y = get_cursor_y();
-// cursor position relative to output
-		float output_x = x;
-		float output_y = y;
-		canvas_to_output(mwindow->edl, 
-				0, 
-				output_x, 
-				output_y);
-
-//printf("CWindowCanvas::test_zoom 1 %f %f\n", x, y);
+	float x, y;
+	float zoom = 0;
 
+	if( mwindow->edl->session->cwindow_scrollbars ) {
+		if( *gui->zoom_panel->get_text() != 'x' ) {
 // Find current zoom in table
-		int current_index = 0;
-		for(current_index = 0 ; current_index < total_zooms; current_index++)
-			if(EQUIV(my_zoom_table[current_index], zoom)) break;
-
-
-// Zoom out
-		if(get_buttonpress() == 5 ||
-			gui->ctrl_down() || 
-			gui->shift_down())
-		{
-			current_index--;
+			int idx = total_zooms;  float old_zoom = get_zoom();
+			while( --idx >= 0 && !EQUIV(my_zoom_table[idx], old_zoom) );
+			if( idx >= 0 ) {
+				idx += get_buttonpress() == 5 ||
+					 gui->ctrl_down() || gui->shift_down() ?  -1 : +1 ;
+				CLAMP(idx, 0, total_zooms-1);
+				zoom = my_zoom_table[idx];
+			}
 		}
-		else
-// Zoom in
-		{
-			current_index++;
+		x = get_cursor_x();  y = get_cursor_y();
+		if( !zoom ) {
+			mwindow->edl->session->cwindow_scrollbars = 0;
+			gui->zoom_panel->update(0);
+			zoom = gui->get_auto_zoom();
+		}
+		else {
+			gui->zoom_panel->ZoomPanel::update(zoom);
+			float output_x = x, output_y = y;
+			canvas_to_output(mwindow->edl, 0, output_x, output_y);
+			x = output_x - x / zoom;
+			y = output_y - y / zoom;
 		}
-		
-		CLAMP(current_index, 0, total_zooms - 1);
-		zoom = my_zoom_table[current_index];
-		
-		x = output_x - x / zoom;
-		y = output_y - y / zoom;
-
-		
 	}
+	else {
+		mwindow->edl->session->cwindow_scrollbars = 1;
+		x = (mwindow->edl->session->output_w - w) / 2;
+		y = (mwindow->edl->session->output_h - h) / 2;
+		zoom = 1;
+	}
+	update_zoom((int)x, (int)y, zoom);
 
+	gui->composite_panel->cpanel_zoom->update(zoom);
 
+	reposition_window(mwindow->edl,
+			mwindow->theme->ccanvas_x, mwindow->theme->ccanvas_y,
+			mwindow->theme->ccanvas_w, mwindow->theme->ccanvas_h);
+	redraw = 1;  result = 1;
 
-	int x_i = (int)x;
-	int y_i = (int)y;
-//	check_boundaries(mwindow->edl, x_i, y_i, zoom);
-
-//printf("CWindowCanvas::test_zoom 2 %d %d\n", x_i, y_i);
-
-	update_zoom(x_i, 
-			y_i, 
-			zoom);
-	reposition_window(mwindow->edl, 
-			mwindow->theme->ccanvas_x,
-			mwindow->theme->ccanvas_y,
-			mwindow->theme->ccanvas_w,
-			mwindow->theme->ccanvas_h);
-	redraw = 1;
-	result = 1;
-
-	
-	gui->zoom_panel->update(zoom);
-	
 	return result;
 }
 
@@ -3106,8 +3074,8 @@ int CWindowCanvas::cursor_motion_event()
 			int x = (int)(gui->x_origin - cursor_x + gui->x_offset);
 			int y = (int)(gui->y_origin - cursor_y + gui->y_offset);
 
-			update_zoom(x, 
-				y, 
+			update_zoom(x,
+				y,
 				zoom);
 			update_scrollbars(0);
 			redraw = 1;
@@ -3130,9 +3098,9 @@ int CWindowCanvas::cursor_motion_event()
 
 		case CWINDOW_CROP:
 			result = test_crop(0, redraw);
-// printf("CWindowCanvas::cursor_motion_event %d result=%d redraw=%d\n", 
-// __LINE__, 
-// result, 
+// printf("CWindowCanvas::cursor_motion_event %d result=%d redraw=%d\n",
+// __LINE__,
+// result,
 // redraw);
 			break;
 
@@ -3141,9 +3109,9 @@ int CWindowCanvas::cursor_motion_event()
 		case CWINDOW_MASK_CONTROL_OUT:
 		case CWINDOW_MASK_TRANSLATE:
 
-			result = do_mask(redraw, 
-				rerender, 
-				0, 
+			result = do_mask(redraw,
+				rerender,
+				0,
 				1,
 				0);
 			break;
@@ -3161,8 +3129,8 @@ int CWindowCanvas::cursor_motion_event()
 // cursor font changes
 	if(!result)
 	{
-// printf("CWindowCanvas::cursor_motion_event %d cwindow_operation=%d\n", 
-// __LINE__, 
+// printf("CWindowCanvas::cursor_motion_event %d cwindow_operation=%d\n",
+// __LINE__,
 // mwindow->edl->session->cwindow_operation);
 		switch(mwindow->edl->session->cwindow_operation)
 		{
@@ -3173,9 +3141,9 @@ int CWindowCanvas::cursor_motion_event()
 				result = do_ruler(0, 1, 0, 0);
 				break;
 			case CWINDOW_MASK:
-				result = do_mask(redraw, 
-					rerender, 
-					0, 
+				result = do_mask(redraw,
+					rerender,
+					0,
 					1,
 					0);
 					break;
@@ -3196,12 +3164,12 @@ int CWindowCanvas::cursor_motion_event()
 	if(redraw_canvas)
 	{
 		gui->unlock_window();
-	
-	
+
+
 		mwindow->gui->lock_window("CWindowCanvas::cursor_motion_event 1");
 		mwindow->gui->draw_overlays(1);
 		mwindow->gui->unlock_window();
-		
+
 		gui->lock_window("CWindowCanvas::cursor_motion_event 1");
 	}
 
@@ -3210,7 +3178,7 @@ int CWindowCanvas::cursor_motion_event()
 		gui->unlock_window();
 		mwindow->restart_brender();
 		mwindow->sync_parameters(CHANGE_PARAMS);
-		gui->cwindow->playback_engine->que->send_command(CURRENT_FRAME, 
+		gui->cwindow->playback_engine->que->send_command(CURRENT_FRAME,
 			CHANGE_NONE,
 			mwindow->edl,
 			1);
@@ -3229,7 +3197,7 @@ int CWindowCanvas::button_press_event()
 
 	if(Canvas::button_press_event()) return 1;
 
-	gui->translating_zoom = gui->shift_down(); 
+	gui->translating_zoom = gui->shift_down();
 
 	calculate_origin();
 //printf("CWindowCanvas::button_press_event 2 %f %f\n", gui->x_origin, gui->y_origin, gui->x_origin, gui->y_origin);
@@ -3253,7 +3221,7 @@ int CWindowCanvas::button_press_event()
 			case CWINDOW_RULER:
 				result = do_ruler(0, 0, 1, 0);
 				break;
-		
+
 			case CWINDOW_CAMERA:
 				result = test_bezier(1, redraw, redraw_canvas, rerender, 1);
 				break;
@@ -3285,24 +3253,24 @@ int CWindowCanvas::button_press_event()
 	{
 		draw_refresh();
 		gui->unlock_window();
-	
-	
+
+
 		mwindow->gui->lock_window("CWindowCanvas::button_press_event 1");
 		mwindow->gui->draw_overlays(1);
 		mwindow->gui->unlock_window();
 		gui->update_tool();
-		
+
 		gui->lock_window("CWindowCanvas::button_press_event 1");
 	}
 
 // rerendering can also be caused by press event
-	if(rerender) 
+	if(rerender)
 	{
 		gui->unlock_window();
 
 		mwindow->restart_brender();
 		mwindow->sync_parameters(CHANGE_PARAMS);
-		gui->cwindow->playback_engine->que->send_command(CURRENT_FRAME, 
+		gui->cwindow->playback_engine->que->send_command(CURRENT_FRAME,
 			CHANGE_NONE,
 			mwindow->edl,
 			1);
@@ -3321,7 +3289,7 @@ int CWindowCanvas::button_release_event()
 		case CWINDOW_SCROLL:
 			result = 1;
 			break;
-			
+
 		case CWINDOW_RULER:
 			do_ruler(0, 0, 0, 1);
 			break;
@@ -3342,7 +3310,9 @@ int CWindowCanvas::button_release_event()
 			gui->mask_keyframe = 0;
 			mwindow->undo->update_undo_after(_("mask"), LOAD_AUTOMATION);
 			break;
-
+		case CWINDOW_NONE:
+			result = Canvas::button_release_event();
+			break;
 	}
 
 	gui->current_operation = CWINDOW_NONE;
@@ -3353,12 +3323,12 @@ void CWindowCanvas::zoom_resize_window(float percentage)
 {
 	int canvas_w, canvas_h;
 	int new_w, new_h;
-	
-	
+
+
 // Get required canvas size
-	calculate_sizes(mwindow->edl->get_aspect_ratio(), 
-		mwindow->edl->session->output_w, 
-		mwindow->edl->session->output_h, 
+	calculate_sizes(mwindow->edl->get_aspect_ratio(),
+		mwindow->edl->session->output_w,
+		mwindow->edl->session->output_h,
 		percentage,
 		canvas_w,
 		canvas_h);
@@ -3371,7 +3341,7 @@ void CWindowCanvas::zoom_resize_window(float percentage)
 	mwindow->session->cwindow_w = new_w;
 	mwindow->session->cwindow_h = new_h;
 
-	mwindow->theme->get_cwindow_sizes(gui, 
+	mwindow->theme->get_cwindow_sizes(gui,
 		mwindow->session->cwindow_controls);
 
 // Estimate again from new borders