Bug 511

Summary:	The view is sometimes not redrawn when necessary
Product:	PulseView	Reporter:	Soeren Apel <soeren>
Component:	Data display	Assignee:	Nobody <nobody>
Status:	CONFIRMED ---
Severity:	normal	CC:	joel
Priority:	Normal
Version:	unreleased development snapshot
Target Milestone:	PulseView 0.3.0
Hardware:	All
OS:	All

Description Soeren Apel 2014-12-02 21:51:43 CET

Problem: In certain situations, the view does not reflect changes that were performed in the UI.

How to replicate:

One of the situations in which this occurs is:
-) Start PV
-) Run demo
-) Add parallel decoder
-) Change decoder name to anything else, e.g. "abc" (note how the decode trace label doesn't update while typing either)
-) Right-click the decoder trace and delete it
Result: decoder trace label is gone, trace itself is still there

This behavior was introduced with the changes following c294543f8df3b3c90fa613c8198506aea68e91e7.

Comment 1 Soeren Apel 2014-12-03 13:08:01 CET

The commit that introduced this bug is 448a72cf7f0225eace2335ec05b979c4e9a6b882.

Comment 2 Joel Holdsworth 2014-12-04 10:40:49 CET

RowItem objects can trigger redraws by invoking appearance_changed( ), (or the top-level view can trigger updates by directly calling update() ). This issue is likely due to neglecting to call one or other of these.