Set sticky_edges correctly for negative height bar().

anntzer · anntzer · commit da7564db421f · 2017-02-15T22:22:31.000-08:00
The patch correctly sets the sticky edge to 0 instead of -1 for
 `bar(0, height=-1)`.
diff --git a/doc/api/api_changes/2017-01-30-AL_negative_bar_height_width.rst b/doc/api/api_changes/2017-01-30-AL_negative_bar_height_width.rst
@@ -0,0 +1,7 @@
+`bar` now returns rectangles of negative height or width if the corresponding input is negative
+```````````````````````````````````````````````````````````````````````````````````````````````
+
+`plt.bar` used to normalize the coordinates of the rectangles that it created,
+to keep their height and width positives, even if the corresponding input was
+negative.  This normalization has been removed to permit a simpler computation
+of the correct `sticky_edges` to use.
diff --git a/lib/matplotlib/axes/_axes.py b/lib/matplotlib/axes/_axes.py
@@ -2114,12 +2114,6 @@ def make_iterable(x):
 
         args = zip(left, bottom, width, height, color, edgecolor, linewidth)
         for l, b, w, h, c, e, lw in args:
-            if h < 0:
-                b += h
-                h = abs(h)
-            if w < 0:
-                l += w
-                w = abs(w)
             r = mpatches.Rectangle(
                 xy=(l, b), width=w, height=h,
                 facecolor=c,
diff --git a/lib/matplotlib/tests/test_axes.py b/lib/matplotlib/tests/test_axes.py
@@ -4388,16 +4388,6 @@ def test_rc_major_minor_tick():
         assert yax._minor_tick_kw['tick2On'] == True
 
 
-def test_bar_negative_width():
-    fig, ax = plt.subplots()
-    res = ax.bar(range(1, 5), range(1, 5), width=-1)
-    assert len(res) == 4
-    for indx, b in enumerate(res):
-        assert b._x == indx
-        assert b._width == 1
-        assert b._height == indx + 1
-
-
 def test_square_plot():
     x = np.arange(4)
     y = np.array([1., 3., 5., 7.])
