Skip to content

Remove unnecessary uses of "use_draw_order"#661

Merged
rahulporuri merged 2 commits into
masterfrom
cln/use_draw_order_usage
Apr 15, 2021
Merged

Remove unnecessary uses of "use_draw_order"#661
rahulporuri merged 2 commits into
masterfrom
cln/use_draw_order_usage

Conversation

@rahulporuri
Copy link
Copy Markdown
Contributor

This PR removes unnecessary uses of use_draw_order in the code - which is by default True - and redefinition of the use_draw_order trait on a PlotComponent subclass and a commented out piece of code.

Note that I tested the zoomed_plot and it still works as expected. You need to do python -m zoomed_plot.zoom_plot if you want to run the example (which I think needs to be documented somewhere).

Related to #659

and redefinition of the trait on a PlotComponent subclass
and a commented out piece of code

	modified:   chaco/barplot.py
	modified:   chaco/color_bar.py
	modified:   examples/demo/zoomed_plot/grid_plot_factory.py
	modified:   examples/demo/zoomed_plot/zoom_plot.py
Copy link
Copy Markdown
Contributor

@aaronayres35 aaronayres35 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rahulporuri rahulporuri merged commit 6bc5b9f into master Apr 15, 2021
@rahulporuri rahulporuri deleted the cln/use_draw_order_usage branch April 15, 2021 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants