Opened 6 years ago

Last modified 5 years ago

#4899 new enhancement

Unify RangeOverlayRenderer and RangeOverlayManager

Reported by: elexis Owned by:
Priority: Nice to Have Milestone: Backlog
Component: Simulation Keywords:
Cc: Patch:

Description

In ​Phab:D238 we have added the RangeVisualization JS component and used the Selectable component to render range overlays.

In ​Phab:D555 we have moved the rendering from Selectable to a new RangeOverlayRenderer component.

r20624 simplified the code a bit, so with a bit more of polishing, the two components might become merged again in one C++ component (unless there is some catch).

Change History (2)

comment:1 by elexis, 6 years ago

In 20627:

Rename RangeVisualization component from rP19519 / D238 to RangeOverlayManager for consistency with the RangeOverlayRenderer from rP20622 / D555.
Refs #4899, #3915

comment:2 by Itms, 5 years ago

Component: UI & Simulation → Simulation
Note: See TracTickets for help on using tickets.