From db45de9bae0840ac04d768c7a5a2928339cf4d6e Mon Sep 17 00:00:00 2001
From: gvasu <gayathriv.86@gmail.com>
Date: Sun, 14 Jul 2013 14:07:33 -0700
Subject: [PATCH 1/3] Automate Work / Break Cycle

Give the users a choice in preferences section (called 'auto mode')
which makes the extension to go in a work / break loop. This removes the
need to manually intervene to flip the work/break switch.

To Do: International strings. I have updated only locales\EN.

Future Improvements:
1) This option wont work well with Notifications. users won't see
notifications  because we
dismiss them on onStart when the next cycle begins. Users may
potentially notice
notification appear and disappear in a flash. This is minor annoyance or
sometimes un-noticeable. We can potentially uncheck Notifications, When
Auto-Mode is set.

2) I haven't addressed when the work / break loop expires. This will go
on in a loop. The user has to uncheck auto-mode option to break out of
the loop.
---
 _locales/en/messages.json |  4 ++++
 background.js             | 10 ++++++++++
 options.html              |  4 ++++
 options.js                |  4 ++++
 4 files changed, 22 insertions(+)

diff --git a/_locales/en/messages.json b/_locales/en/messages.json
index d244cdd..bae82c0 100644
--- a/_locales/en/messages.json
+++ b/_locales/en/messages.json
@@ -70,6 +70,10 @@
     "message": "Ring when a timer finishes (audio)",
     "description": "On the options page, label for the checkbox that enables/disables ringing sound at the end of a timer"
   },
+  "options_auto_mode": {
+    "message": "Alternate between work and break cycles automatically",
+    "description": "On the options page, label for the checkbox that enables/disables automatic flipping between work and break modes"
+  },
   "options_click_restarts": {
     "message": "Clicking on a running timer restarts it",
     "description": "On the options page, label for the checkbox that enables/disables the feature that clicking on a running timer restarts it"
diff --git a/background.js b/background.js
index d09ff79..ddec5f6 100644
--- a/background.js
+++ b/background.js
@@ -318,6 +318,16 @@ var notification, mainPomodoro = new Pomodoro({
         console.log("playing ring", RING);
         RING.play();
       }
+      
+      // If auto-mode is set, restart the cycle by calling start
+      // Note:  If this is set, users won't see notifications  because we  
+      //        dismissthem on onStart at Line 344. Users may potentially notice
+      //        notification appear and disappear in a flash.
+      // To Do: International strings      
+      if(PREFS.autoMode) {
+         mainPomodoro.start();
+      }
+      
     },
     onStart: function (timer) {
       chrome.browserAction.setIcon({
diff --git a/options.html b/options.html
index d97ca87..638d9b2 100644
--- a/options.html
+++ b/options.html
@@ -135,6 +135,10 @@ <h1 data-i18n="options_title"></h1>
         <input id="should-ring" type="checkbox" />
         <label for="should-ring" data-i18n="options_should_ring"></label>
       </div>
+      <div>
+        <input id="auto-mode" type="checkbox" />
+        <label for="auto-mode" data-i18n="options_auto_mode"></label>
+      </div>
       <div>
         <input id="click-restarts" type="checkbox" />
         <label for="click-restarts">
diff --git a/options.js b/options.js
index 32dada8..e8efe2c 100644
--- a/options.js
+++ b/options.js
@@ -25,6 +25,7 @@ var form = document.getElementById('options-form'),
   whitelistEl = document.getElementById('blacklist-or-whitelist'),
   showNotificationsEl = document.getElementById('show-notifications'),
   shouldRingEl = document.getElementById('should-ring'),
+  autoModeEl =  document.getElementById('auto-mode'),
   clickRestartsEl = document.getElementById('click-restarts'),
   saveSuccessfulEl = document.getElementById('save-successful'),
   timeFormatErrorEl = document.getElementById('time-format-error'),
@@ -62,6 +63,7 @@ form.onsubmit = function () {
     durations:          durations,
     showNotifications:  showNotificationsEl.checked,
     shouldRing:         shouldRingEl.checked,
+    autoMode:           autoModeEl.checked,
     clickRestarts:      clickRestartsEl.checked,
     whitelist:          whitelistEl.selectedIndex == 1
   })
@@ -72,6 +74,7 @@ form.onsubmit = function () {
 siteListEl.onfocus = formAltered;
 showNotificationsEl.onchange = formAltered;
 shouldRingEl.onchange = formAltered;
+autoModeEl.onchange = formAltered;
 clickRestartsEl.onchange = formAltered;
 whitelistEl.onchange = formAltered;
 
@@ -83,6 +86,7 @@ function formAltered() {
 siteListEl.value = background.PREFS.siteList.join("\n");
 showNotificationsEl.checked = background.PREFS.showNotifications;
 shouldRingEl.checked = background.PREFS.shouldRing;
+autoModeEl.checked = background.PREFS.autoMode;
 clickRestartsEl.checked = background.PREFS.clickRestarts;
 whitelistEl.selectedIndex = background.PREFS.whitelist ? 1 : 0;
 

From e85fdbc2370853ddcac9cce51151759b0a754db1 Mon Sep 17 00:00:00 2001
From: balajikris <balaji.krishnan@gmail.com>
Date: Sun, 14 Jul 2013 18:28:09 -0700
Subject: [PATCH 2/3] auto mode update

Wait 30 seconds before switching between work and break modes
automatically. I basically queue up a start(), which executes 30 seconds
later.
If the user tries to manually intervene the switching in auto mode, bad
things can happen - the queued up start will reset the timer which was
manually triggered by the user. this is less than desirable but I'm not
worrying about fixing it now.
---
 background.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/background.js b/background.js
index ddec5f6..554ca64 100644
--- a/background.js
+++ b/background.js
@@ -325,7 +325,7 @@ var notification, mainPomodoro = new Pomodoro({
       //        notification appear and disappear in a flash.
       // To Do: International strings      
       if(PREFS.autoMode) {
-         mainPomodoro.start();
+         setTimeout(function(){mainPomodoro.start();}, 30000);
       }
       
     },

From b3835b32d15fac42f76a19900e410e5efaf5af86 Mon Sep 17 00:00:00 2001
From: balajikris <balaji.krishnan@gmail.com>
Date: Sun, 14 Jul 2013 18:30:46 -0700
Subject: [PATCH 3/3] remove comment

remove a comment that is no longer applicable
---
 background.js | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/background.js b/background.js
index 554ca64..b7f8d05 100644
--- a/background.js
+++ b/background.js
@@ -320,9 +320,6 @@ var notification, mainPomodoro = new Pomodoro({
       }
       
       // If auto-mode is set, restart the cycle by calling start
-      // Note:  If this is set, users won't see notifications  because we  
-      //        dismissthem on onStart at Line 344. Users may potentially notice
-      //        notification appear and disappear in a flash.
       // To Do: International strings      
       if(PREFS.autoMode) {
          setTimeout(function(){mainPomodoro.start();}, 30000);