Skip to content

Commit c664371

Browse files
committed
Reword auto-zooming code comprehension for puny hu-mans.
1 parent 193c0b5 commit c664371

1 file changed

Lines changed: 22 additions & 9 deletions

File tree

public/javascripts/DV/helpers/helpers.js

Lines changed: 22 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -480,20 +480,33 @@ DV.Schema.helpers = {
480480
}
481481

482482
// Setup ranges for auto-width zooming
483+
// A document's zoom slider scales depending on the size of the viewer
484+
// and the default zoom level
483485
var ranges = [];
484486
if (zoom <= 500) {
485-
var zoom2 = (zoom + 700) / 2;
486-
ranges = [zoom, zoom2, 700, 850, 1000];
487+
var medium = 700;
488+
var smallest = zoom;
489+
var small = (smallest + medium) / 2; // avg between medium and smallest
490+
ranges = [smallest, small, medium, 850, 1000];
487491
} else if (zoom <= 750) {
488-
var zoom2 = ((1000 - 700) / 3) + zoom;
489-
var zoom3 = ((1000 - 700) / 3)*2 + zoom;
490-
ranges = [.66*zoom, zoom, zoom2, zoom3, 1000];
492+
var small = zoom;
493+
var smallest = 0.66*small;
494+
var medium = ((1000 - 700) / 3) + small; // ???
495+
var large = ((1000 - 700) / 3)*2 + small;
496+
ranges = [smallest, small, medium, large, 1000];
491497
} else if (750 < zoom && zoom <= 850){
492-
var zoom2 = ((1000 - zoom) / 2) + zoom;
493-
ranges = [.66*zoom, 700, zoom, zoom2, 1000];
498+
var small = 700;
499+
var largest = 1000;
500+
var medium = zoom;
501+
var smallest = 0.66*medium;
502+
var large = ((largest - medium) / 2) + medium; // midpoint between medium and largest
503+
ranges = [smallest, small, medium, large, largest];
494504
} else if (850 < zoom && zoom < 1000){
495-
var zoom2 = ((zoom - 700) / 2) + 700;
496-
ranges = [.66*zoom, 700, zoom2, zoom, 1000];
505+
var large = zoom;
506+
var smallest = 0.66*large;
507+
var small = 700;
508+
var medium = ((large - small) / 2) + small; // midpoint between large and small
509+
ranges = [smallest, small, medium, large, 1000];
497510
} else if (zoom >= 1000) {
498511
zoom = 1000;
499512
ranges = this.viewer.models.document.ZOOM_RANGES;

0 commit comments

Comments
 (0)