Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consolidate overlays into "chaco.overlays"? #699

Closed
rahulporuri opened this issue Apr 20, 2021 · 1 comment · Fixed by #741
Closed

Consolidate overlays into "chaco.overlays"? #699

rahulporuri opened this issue Apr 20, 2021 · 1 comment · Fixed by #741

Comments

@rahulporuri
Copy link
Contributor

class PlotAxis(AbstractOverlay):

class ColormappedSelectionOverlay(AbstractOverlay):

class PlotGrid(AbstractOverlay):

class LassoOverlay(AbstractOverlay):

class Legend(AbstractOverlay):

class PlotLabel(AbstractOverlay):

class ScatterInspectorOverlay(AbstractOverlay):

class TextBoxOverlay(AbstractOverlay):

class ToolTip(AbstractOverlay):

Q. What about layers?

class StatusLayer(AbstractOverlay):

@corranwebster
Copy link
Contributor

Axes and grids are a little trickier, since they have a lot of machinery behind them, so you'd either want them to be separate, or to possibly bring in all the scale and tick generation code as well.

Layers probably should go in overlays as well - I think they may have been separated out before there was an overlays directory, but they are just a special case of overlays.

@rahulporuri rahulporuri changed the title Consolidate "AbstractOverlay"s into "chaco.overlays"? Consolidate overlays into "chaco.overlays"? May 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants