Skip to content

Call GetCurrentPoint as intended#268

Merged
jwiggins merged 1 commit into
masterfrom
fix/silly-savage-bug
Feb 8, 2017
Merged

Call GetCurrentPoint as intended#268
jwiggins merged 1 commit into
masterfrom
fix/silly-savage-bug

Conversation

@jwiggins

@jwiggins jwiggins commented Feb 7, 2017

Copy link
Copy Markdown
Member

Fixes #267

The CompiledPath class in the kiva savage backend already had a workaround for the get_current_point method being absent. It only needed to be connected.

The CompiledPath class in the kiva SVG renderer backend already had a
workaround for the `get_current_point` method being absent. It only
needed to be connected.
@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #268 into master will not impact coverage by -18.09%.

@@             Coverage Diff             @@
##           master     #268       +/-   ##
===========================================
- Coverage   53.53%   35.44%   -18.09%     
===========================================
  Files         119      205       +86     
  Lines       12446    18799     +6353     
  Branches     1787     2627      +840     
===========================================
+ Hits         6663     6664        +1     
- Misses       5358    11700     +6342     
- Partials      425      435       +10
Impacted Files Coverage Δ
enable/savage/svg/backends/kiva/renderer.py 30.28% <ø> (+0.65%)
enable/constraints_container.py 55.74% <ø> (-25.63%)
enable/layout/layout_helpers.py 50.68% <ø> (-20.6%)
enable/layout/linear_symbolic.py 87.5% <ø> (-12.5%)
enable/wx/scrollbar.py 33.87% <ø> (ø)
enable/text_grid.py 0% <ø> (ø)
enable/drawing/drawing_canvas.py 0% <ø> (ø)
enable/controls.py 0% <ø> (ø)
enable/trait_defs/ui/rgba_color_editor.py 0% <ø> (ø)
enable/wx/gl.py 0% <ø> (ø)
... and 81 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 04d82e0...5c99369. Read the comment docs.

@rkern

rkern commented Feb 8, 2017

Copy link
Copy Markdown
Member

LGTM.

@jwiggins jwiggins merged commit 359c2f9 into master Feb 8, 2017
@jwiggins

jwiggins commented Feb 8, 2017

Copy link
Copy Markdown
Member Author

Thanks for reviewing!

@jwiggins jwiggins deleted the fix/silly-savage-bug branch February 8, 2017 07:52
mdickinson pushed a commit that referenced this pull request May 8, 2017
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.

3 participants