round dtick 2 to 7 on rangebreaks with day of week pattern

plotly · Mar 27, 2020 · c340efc · c340efc
1 parent 7ad4317
commit c340efc
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 5 deletions.
diff --git a/src/plots/cartesian/axes.js b/src/plots/cartesian/axes.js
@@ -807,11 +807,7 @@ axes.autoTicks = function(ax, roughDTick) {
             ax.dtick = 'M' + roundDTick(roughDTick, 1, roundBase24);
         } else if(roughX2 > ONEDAY) {
             ax.dtick = roundDTick(roughDTick, ONEDAY, roundDays);
-
-            if(ax._hasDayOfWeekBreaks) {
-                if(ax.dtick === 2 * ONEDAY) ax.dtick = ONEDAY;
-                else if(ax.dtick === 3 * ONEDAY) ax.dtick = 7 * ONEDAY;
-            }
+            if(ax._hasDayOfWeekBreaks && ax.dtick < 7 * ONEDAY) ax.dtick = 7 * ONEDAY;
 
             // get week ticks on sunday
             // this will also move the base tick off 2000-01-01 if dtick is

diff --git a/test/image/baselines/axes_breaks-round-weekdays.png b/test/image/baselines/axes_breaks-round-weekdays.png
diff --git a/test/image/baselines/axes_breaks-weekends_autorange-reversed.png b/test/image/baselines/axes_breaks-weekends_autorange-reversed.png