0003027: Scalarbar label opacity, font and style do not update in colors module - MantisBT

ID	Project	Category	View Status	Date Submitted	Last Update

0003027	Slicer4	Core: GUI	public	2013-03-22 09:30	2014-03-06 06:16

Reporter	wangk	Assigned To	nicole
Priority	normal	Severity	minor	Reproducibility	always
Status	closed	Resolution	fixed
Product Version	Slicer 4.2.2-1
Target Version	Slicer 4.3.0	Fixed in Version	Slicer 4.3.0

Summary	0003027: Scalarbar label opacity, font and style do not update in colors module
Description	(From Nicole) Label color, opacity, font, style all require a click in the 3D window to show the updated setting. The problem could either be in the CTK widget when it updates the setting but doesn't trigger an event, or in the way the VTK widget is hooked up to the 3D window (the dipslayable manager not listening for certain events?)
Tags	No tags attached.

nicole 2013-03-25 12:16 administrator ~0008234	Looks like the problem is that the ctkVTKScalarBarWidget has a ctkVTKTextPropertyWidget embedded within it, and is only listening to the changes in the label format. There's no accessor for the property widget and without that I can't get at the textProperty() to listen for VTK modified events. Looking at a few possible solutions in CTK and will submit a pull request there if necessary and link back here.

nicole 2013-03-25 14:41 administrator ~0008240 Last edited: 2013-03-26 10:34	Seemed easiest to modify the text property widget to emit a modified() signal and have the scalar bar widget pass it along. That way Slicer will see the modified signal and request a render. https://github.com/naucoin/CTK/commit/3027-ScalarBar-Respond-To-Updates-From-Labels-Text-Property-Widget Submitting a pull request to CTK: https://github.com/commontk/CTK/pull/311

nicole 2013-04-05 12:29 administrator ~0008324	Updated code after comments to use multiple signals instead of just one modified: https://github.com/naucoin/CTK/commit/042b601b5ecb335e646aea462b2be231293a7fd8

nicole 2013-04-30 06:41 administrator ~0008541	Pull request merged by Julien: https://github.com/commontk/CTK/pull/311

nicole 2014-02-03 12:12 administrator ~0010553	Updated code when realised that the title text property changes weren't being monitored (previous fix only listened for signals on the label text property). https://github.com/naucoin/CTK/tree/3027-ScalarBar-Respond-To-Updates-From-Title-Text-Property-Widget Pull request: https://github.com/commontk/CTK/pull/436

nicole 2014-02-03 13:20 administrator ~0010554	Pull request 436 merged by Julien.

Date Modified	Username	Field	Change
2013-03-22 09:30	wangk	New Issue
2013-03-22 09:30	wangk	Status	new => assigned
2013-03-22 09:30	wangk	Assigned To	=> kikinis
2013-03-22 09:31	wangk	Relationship added	related to 0002919
2013-03-22 18:09	kikinis	Assigned To	kikinis => nicole
2013-03-25 12:16	nicole	Note Added: 0008234
2013-03-25 14:41	nicole	Note Added: 0008240
2013-03-26 10:34	nicole	Note Edited: 0008240
2013-04-05 12:29	nicole	Note Added: 0008324
2013-04-30 06:41	nicole	Note Added: 0008541
2013-09-03 12:22	nicole	Status	assigned => resolved
2013-09-03 12:22	nicole	Fixed in Version	=> Slicer 4.3.0
2013-09-03 12:22	nicole	Resolution	open => fixed
2014-02-03 12:12	nicole	Note Added: 0010553
2014-02-03 13:20	nicole	Note Added: 0010554
2014-02-28 06:27	wangk	Status	resolved => closed
2014-03-06 06:16	jcfr	Target Version	=> Slicer 4.3.0