change_exposure(): Pull exposure logic together

diff --git a/picsorter.js b/picsorter.js
index d721ac4eeae7e93a31cbeb3ba528b2665e54dace..808855bcbb83477dc76d0e7bf93b40f1dd5eeecd 100644 (file)
--- a/picsorter.js
+++ b/picsorter.js
@@ -23,11 +23,6 @@ function say(message) {
   setTimeout(function() { $("#message").addClass("fade"); }, 1);
 }
 
   setTimeout(function() { $("#message").addClass("fade"); }, 1);
 }
 

-function say_exposure() {
-  var e = (exposure / 4) - 3.5;
-  say((e >= 0 ? "+" : "") + e);
-}
-

 function toggle_zoom() {
   if (zoom) {
     zoom = "";
 function toggle_zoom() {
   if (zoom) {
     zoom = "";


@@ -57,6 +52,13 @@ function mark_not_deleted() {
   say("Undeleted");
 }
 
   say("Undeleted");
 }
 

+function change_exposure(amount) {
+  exposure += amount;
+  setpic();
+  var display_exposure = (exposure / 4) - 3.5;
+  say((display_exposure >= 0 ? "+" : "") + display_exposure);
+}
+

 $(function() {
   move_to_nondeleted(1);
 });
 $(function() {
   move_to_nondeleted(1);
 });


@@ -67,7 +69,7 @@ Mousetrap.bind('n', function() { move_to_nondeleted(1); });
 Mousetrap.bind('p', function() { move_to_nondeleted(-1); });
 Mousetrap.bind('N', function() { input_index ++; say(input_index); setpic(); });
 Mousetrap.bind('P', function() { input_index --; say(input_index); setpic(); });
 Mousetrap.bind('p', function() { move_to_nondeleted(-1); });
 Mousetrap.bind('N', function() { input_index ++; say(input_index); setpic(); });
 Mousetrap.bind('P', function() { input_index --; say(input_index); setpic(); });

-Mousetrap.bind('b', function() { exposure ++; say_exposure(); setpic(); });
-Mousetrap.bind('d', function() { exposure --; say_exposure(); setpic(); });
+Mousetrap.bind('b', function() { change_exposure(1); });
+Mousetrap.bind('d', function() { change_exposure(-1); });

 Mousetrap.bind('x', mark_deleted);
 Mousetrap.bind('X', mark_not_deleted);
 Mousetrap.bind('x', mark_deleted);
 Mousetrap.bind('X', mark_not_deleted);