Skip to content

Add support for view node visibility

Sam Horvath requested to merge github/fork/jcfr/add-support-for-view-node-visibility into master

Created by: jcfr

After integrating this topic, calling SetVisibility() on a view node will effectively hide/show the associated widget in the layout.

Note that as detailed in 451973b, this set of patches disable the reading of view node visibility attribute from the MRML file. Associated comment is also reported below:

+    // XXX Do not read 'visibility' attribute and default to 1 because:
+    // (1) commit r21034 (STYLE: Add abstract class for all view nodes)
+    // changed the default value for 'visibility' attribute from 1 to 0. This
+    // means there are a lot of already saved scene where visibility attribute
+    // value is saved as 0.
+    // (2) support for visibility attribute by the layout manager has been
+    // added.
+    // XXX Support for 'visibility' attribute could be restored by updating
+    // the mrml version. Scene with a newer version number would consider the
+    // serialized attribute whereas older scene would not.

Merge request reports