Ticket #3958 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Make VATES views handle sources better

Reported by: Michael Reuter Owned by: Michael Reuter
Priority: major Milestone: Release 2.0
Component: VATES Keywords:
Cc: Blocked By: #3954
Blocking: #3929 Tester: Owen Arnold

Description

The VATES simple interface views should be passed pipeline sources from the main widget and each view should internally handle how to deal with the source.

Change History

comment:1 Changed 9 years ago by Michael Reuter

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Michael Reuter

In [15272]:

Set workspace type just once. This refs #3958.

comment:3 Changed 9 years ago by Michael Reuter

In [15273]:

Revert change since it needs to be done at a different level. This refs #3958.

comment:4 Changed 9 years ago by Michael Reuter

  • Blocking 3929 added

comment:12 Changed 9 years ago by Michael Reuter

In [15315]:

Making sure workspace type property is available from plugin mode. This refs #3958.

comment:13 Changed 9 years ago by Michael Reuter

In [15327]:

Starting to add simplifying functions. This refs #3958.

comment:14 Changed 9 years ago by Michael Reuter

In [15331]:

Put old methodology back for now. This refs #3958.

comment:15 Changed 9 years ago by Michael Reuter

In [15343]:

More internal functions going in. This refs #3958.

comment:16 Changed 9 years ago by Michael Reuter

In [15344]:

Name renormalization. This refs #3958.

comment:10 Changed 9 years ago by Michael Reuter

Using active representation for color handling. This refs #3958.

Changeset: 02d2f97e647c8cfe0fd5361e20f2dab3091d1abf

comment:11 Changed 9 years ago by Michael Reuter

Removing unneeded variable. This refs #3958.

Changeset: c4daf9e078dd1e862b0dfd2e757b588a4c27e267

comment:12 Changed 9 years ago by Michael Reuter

Next step in overlay capability. This refs #3958.

In order to make it easier for the overlay capability to happen, I had to make the plugin mode destroy the current sources when then view window is hidden. This required an eventfilter due to the free floating nature of the view window. For the plugin mode, this change opens up overlay capability. Now the individual view need to handle it.

Changeset: 1652ebb63e837d30e1531eadac1a28ee0ba6c9af

comment:13 Changed 9 years ago by Michael Reuter

Adding some documentation. This refs #3958.

Changeset: 926868223b90c1bb08e52bce7b3beb4286fcb8c4

comment:14 Changed 9 years ago by Michael Reuter

More documentation. This refs #3958.

Changeset: 0f2b390a23a17ddf3b234f5cafde26d205fc5fb8

comment:15 Changed 9 years ago by Michael Reuter

First step in making views do things for themselves. This refs #3958.

Changeset: d21697fb135679ee4e65253f9723a4e624925b30

comment:16 Changed 9 years ago by Michael Reuter

Removing unneeded variable. This refs #3958.

Changeset: c4daf9e078dd1e862b0dfd2e757b588a4c27e267

comment:17 Changed 9 years ago by Michael Reuter

Next step in overlay capability. This refs #3958.

In order to make it easier for the overlay capability to happen, I had to make the plugin mode destroy the current sources when then view window is hidden. This required an eventfilter due to the free floating nature of the view window. For the plugin mode, this change opens up overlay capability. Now the individual view need to handle it.

Changeset: 1652ebb63e837d30e1531eadac1a28ee0ba6c9af

comment:18 Changed 9 years ago by Michael Reuter

Adding some documentation. This refs #3958.

Changeset: 926868223b90c1bb08e52bce7b3beb4286fcb8c4

comment:19 Changed 9 years ago by Michael Reuter

More documentation. This refs #3958.

Changeset: 0f2b390a23a17ddf3b234f5cafde26d205fc5fb8

comment:20 Changed 9 years ago by Michael Reuter

  • Status changed from accepted to verify
  • Resolution set to fixed

If the testing in ticket #3929 works, then this should work since it was the fundamental fix that allows overlays.

comment:21 Changed 9 years ago by Jean Bilheux

  • Status changed from verify to verifying
  • Tester set to Jean Bilheux

comment:22 Changed 9 years ago by Jean Bilheux

  • Status changed from verifying to verify
  • Tester Jean Bilheux deleted

comment:23 Changed 9 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester set to Owen Arnold

comment:24 Changed 9 years ago by Owen Arnold

  • Status changed from verifying to closed

As suggested, verified #3929 first. This seems to works as expected.

comment:25 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 4805

Note: See TracTickets for help on using tickets.