From d1309d5a15f015c52ff48ea52b028f47ca7723e8 Mon Sep 17 00:00:00 2001 From: Scott Worley Date: Thu, 27 Feb 2014 23:15:59 -0800 Subject: [PATCH] Keep a local picsorter_deleted too. Don't unpack it from localStorage every single time it is needed. Read it in once and write it back when it changes. Note: This would make updates from multiple tabs running picsorter on the same domain clobber each other's updates. So don't run multiple picsorters. --- picsorter.js | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/picsorter.js b/picsorter.js index c3af639..e44c22c 100644 --- a/picsorter.js +++ b/picsorter.js @@ -5,6 +5,7 @@ var zoom = "sm/"; if (!("picsorter_deleted" in localStorage)) { localStorage.picsorter_deleted = JSON.stringify({}); } +var picsorter_deleted = JSON.parse(localStorage.picsorter_deleted); function setpic() { $("#pic").attr("src", zoom + exposure + "/" + files[input_index]); @@ -19,22 +20,19 @@ function toggle_zoom() { } function move_to_nondeleted(direction) { - var deleted = JSON.parse(localStorage.picsorter_deleted); do { input_index += direction; - } while (files[input_index] in deleted); + } while (files[input_index] in picsorter_deleted); } function mark_deleted() { - var deleted = JSON.parse(localStorage.picsorter_deleted); - deleted[files[input_index]] = 1; - localStorage.picsorter_deleted = JSON.stringify(deleted); + picsorter_deleted[files[input_index]] = 1; + localStorage.picsorter_deleted = JSON.stringify(picsorter_deleted); } function mark_not_deleted() { - var deleted = JSON.parse(localStorage.picsorter_deleted); - delete deleted[files[input_index]]; - localStorage.picsorter_deleted = JSON.stringify(deleted); + delete picsorter_deleted[files[input_index]]; + localStorage.picsorter_deleted = JSON.stringify(picsorter_deleted); } $(function() { -- 2.44.1