wsphillips
Fix PlotBars() and PlotBarsH() wrappers
Fixes #31. The wrappers for PlotErrorBars() and PlotErrorBarsH() are probably still broken though, will need to fix that later.
@sairus7, do you have merge privs? If so could I nerd-snipe you into reviewing and merging this? :sweat_smile:
Looks like there is conflicting wrapper keyword arguments with previous version. So this is a break change, right?
And also, do we really want to add another wrapper layer of different function signatures, just to make it shorter by one single flag ImPlotBarsFlags_Horizontal / ImPlotBarsFlags_None?
Yes, this would be a breaking change.
And also, do we really want to add another wrapper layer of different function signatures, just to make it shorter by one single flag ImPlotBarsFlags_Horizontal / ImPlotBarsFlags_None?
Well, I didn't add any wrappers here :) If you feel it's simpler to remove PlotBarsH() then I think that's also fine.
I plan to change signatures closer to original implot functions (for example, all functions should have string ID as first argument), but let's leave it for another PR (see #34), as with PlotErrorBarsH(). For now its working and looks good. Merging.