Usually the upper and lower shadow have the same color as the body. The other option is to draw the body first to avoid the ugly black line through it.
But: If the background is black, the shadows will vanish.

If you are going to do work on thefinance.py could you do it againsthttps://github.com/tacaswell/matplotlib/tree/ochl_to_ohlc (see PR#1920 ) to make the eventual merge easier?

If you actuallyusefinance do you want to take over herding#1920 through merging?

@profholzer: Just so we don't lose track of this... Did you close because this is the wrong approach, or because you, following@tacaswell's suggestion, made this against his branch? If the latter, we should reference which PR that work moved to.

Hi,
this was my first attempt to give something back to the matplotlib developers and I simply don't know, what I should do exactly, because I am not familiar to the methods of reporting bugs or enhancements.
Sorry for my inactivity due to the wrong approach. I saw that other users made the same recommendation earlier.

Regards
Alexander

Am 16.07.2013 um 18:43 schrieb Michael Droettboomnotifications@github.com:

@profholzer: Just so we don't lose track of this... Did you close because this is the wrong approach, or because you, following@tacaswell's suggestion, made this against his branch? If the latter, we should reference which PR that work moved to.

—
Reply to this email directly or view it on GitHub.

Ok -- why don't we do this, then. I'll reopen the bug, and once#1902 is merged, we can reassess where we are wrt this issue.

@mdboom There is a typo in your comment, it is#1920 not 1902