X-Git-Url: https://sigrok.org/gitweb/?p=pulseview.git;a=blobdiff_plain;f=pv%2Fview%2Fview.cpp;h=d58e074a3ef72ec77bc3e75c3487dac627cbb15d;hp=083d02dad9c6fa6043ae3c906363165a2c8294c0;hb=98cfe4e8dadad2cf710eb46fd5c85d9d0520a875;hpb=5a3e53f6609ade7eb232ebe46ecb9e66fac09086;ds=sidebyside

diff --git a/pv/view/view.cpp b/pv/view/view.cpp
index 083d02da..d58e074a 100644
--- a/pv/view/view.cpp
+++ b/pv/view/view.cpp
@@ -121,11 +121,6 @@ View::View(Session &session, QWidget *parent) :
 	connect(&session_, SIGNAL(frame_ended()),
 		this, SLOT(data_updated()));
 
-	connect(cursors_->first().get(), SIGNAL(time_changed()),
-		this, SLOT(marker_time_changed()));
-	connect(cursors_->second().get(), SIGNAL(time_changed()),
-		this, SLOT(marker_time_changed()));
-
 	connect(header_, SIGNAL(signals_moved()),
 		this, SLOT(on_signals_moved()));
 
@@ -483,11 +478,11 @@ void View::update_scroll()
 
 	// Set the vertical scrollbar
 	verticalScrollBar()->setPageStep(areaSize.height());
+	verticalScrollBar()->setSingleStep(areaSize.height() / 8);
 
 	const pair<int, int> extents = v_extents();
-	const int extra_scroll_height = (extents.second - extents.first) / 4;
-	verticalScrollBar()->setRange(extents.first - extra_scroll_height,
-		extents.first + extra_scroll_height);
+	verticalScrollBar()->setRange(extents.first - (areaSize.height() / 2),
+		extents.second - (areaSize.height() / 2));
 }
 
 void View::update_layout()
@@ -607,7 +602,7 @@ void View::resizeEvent(QResizeEvent*)
 	update_layout();
 }
 
-void View::appearance_changed(bool label, bool content)
+void View::row_item_appearance_changed(bool label, bool content)
 {
 	if (label)
 		header_->update();
@@ -615,11 +610,19 @@ void View::appearance_changed(bool label, bool content)
 		viewport_->update();
 }
 
+void View::time_item_appearance_changed(bool label, bool content)
+{
+	if (label)
+		cursorheader_->update();
+	if (content)
+		viewport_->update();
+}
+
 void View::extents_changed(bool horz, bool vert)
 {
 	sticky_events_ |=
-		(horz ? SelectableItemHExtentsChanged : 0) |
-		(vert ? SelectableItemVExtentsChanged : 0);
+		(horz ? RowItemHExtentsChanged : 0) |
+		(vert ? RowItemVExtentsChanged : 0);
 	lazy_event_handler_.start();
 }
 
@@ -728,12 +731,6 @@ void View::data_updated()
 	viewport_->update();
 }
 
-void View::marker_time_changed()
-{
-	cursorheader_->update();
-	viewport_->update();
-}
-
 void View::on_signals_moved()
 {
 	update_scroll();
@@ -742,9 +739,9 @@ void View::on_signals_moved()
 
 void View::process_sticky_events()
 {
-	if (sticky_events_ & SelectableItemHExtentsChanged)
+	if (sticky_events_ & RowItemHExtentsChanged)
 		update_layout();
-	if (sticky_events_ & SelectableItemVExtentsChanged)
+	if (sticky_events_ & RowItemVExtentsChanged)
 		restack_all_row_items();
 
 	// Clear the sticky events