X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;ds=sidebyside;f=gtk2_ardour%2Feditor_cursors.cc;h=a820b5fc1fa9a916d0dd35d9ae76d196d1ae45a4;hb=ecb92f474f8f39e04f2aee02dabab7fa03af6c19;hp=557120c1db783b54fcd799816ab6d2fe8549c56b;hpb=3031a99364333b9adcc52c4922687477a1e62337;p=ardour.git

diff --git a/gtk2_ardour/editor_cursors.cc b/gtk2_ardour/editor_cursors.cc
index 557120c1db..a820b5fc1f 100644
--- a/gtk2_ardour/editor_cursors.cc
+++ b/gtk2_ardour/editor_cursors.cc
@@ -33,29 +33,21 @@ using namespace Gtk;
 
 EditorCursor::EditorCursor (Editor& ed, bool (Editor::*callbck)(GdkEvent*,ArdourCanvas::Item*))
 	: _editor (ed)
-	, _time_bars_canvas_item (_editor._time_bars_canvas->root ())
-	, _track_canvas_item (_editor._track_canvas->root ())
+	, _track_canvas_item (new ArdourCanvas::Arrow (_editor.get_hscroll_group()))
 	, _length (1.0)
 {
-	CANVAS_DEBUG_NAME ((&_time_bars_canvas_item), "timebars editor cursor");
-	CANVAS_DEBUG_NAME ((&_track_canvas_item), "track canvas editor cursor");
+	CANVAS_DEBUG_NAME (_track_canvas_item, "track canvas editor cursor");
 
-	_time_bars_canvas_item.set_show_head (0, true);
-	_time_bars_canvas_item.set_head_height (0, 9);
-	_time_bars_canvas_item.set_head_width (0, 16);
-	_time_bars_canvas_item.set_head_outward (0, false);
-	_time_bars_canvas_item.set_show_head (1, false); // head only
-	_time_bars_canvas_item.set_outline_width (0.5);
+	_track_canvas_item->set_show_head (0, true);
+	_track_canvas_item->set_head_height (0, 9);
+	_track_canvas_item->set_head_width (0, 16);
+	_track_canvas_item->set_head_outward (0, false);
+	_track_canvas_item->set_show_head (1, false); // head only
+	_track_canvas_item->set_data ("cursor", this);
 
-	_time_bars_canvas_item.set_data ("cursor", this);
-	_track_canvas_item.set_data ("cursor", this);
-	_track_canvas_item.set_outline_width (0.5);
+	_track_canvas_item->Event.connect (sigc::bind (sigc::mem_fun (ed, callbck), _track_canvas_item));
 
-	_time_bars_canvas_item.Event.connect (sigc::bind (sigc::mem_fun (ed, callbck), &_time_bars_canvas_item));
-	_track_canvas_item.Event.connect (sigc::bind (sigc::mem_fun (ed, callbck), &_track_canvas_item));
-
-	_time_bars_canvas_item.set_y1 (ArdourCanvas::COORD_MAX);
-	_track_canvas_item.set_y1 (ArdourCanvas::COORD_MAX);
+	_track_canvas_item->set_y1 (ArdourCanvas::COORD_MAX);
 	
 	_current_frame = 1; /* force redraw at 0 */
 }
@@ -70,40 +62,29 @@ EditorCursor::set_position (framepos_t frame)
 {
 	PositionChanged (frame);
 
-	/* See Cairo FAQ question on single pixel lines to understand
-	   why we add 0.5
-	*/
-
-	double const new_pos = _editor.sample_to_pixel (frame) + 0.5;
+	double const new_pos = _editor.sample_to_pixel_unrounded (frame);
 
-	if (new_pos != _time_bars_canvas_item.x ()) {
-		_time_bars_canvas_item.set_x (new_pos);
+	if (new_pos != _track_canvas_item->x ()) {
+		_track_canvas_item->set_x (new_pos);
 	}
 
-	if (new_pos != _track_canvas_item.x0 ()) {
-		_track_canvas_item.set_x (new_pos, new_pos);
-	}
-	
 	_current_frame = frame;
 }
 
 void
 EditorCursor::show ()
 {
-	_time_bars_canvas_item.show ();
-	_track_canvas_item.show ();
+	_track_canvas_item->show ();
 }
 
 void
 EditorCursor::hide ()
 {
-	_time_bars_canvas_item.hide ();
-	_track_canvas_item.hide ();
+	_track_canvas_item->hide ();
 }
 
 void
 EditorCursor::set_color (ArdourCanvas::Color color)
 {
-	_time_bars_canvas_item.set_color (color);
-	_track_canvas_item.set_outline_color (color);
+	_track_canvas_item->set_color (color);
 }