Fix long breaks in hours

Author: slgobinath

README.md

@@ -72,8 +72,8 @@ Just install and forget; Safe Eyes will take care of your eyes. To customize the
 
 ### Override individual break time
 
-Add the optional `time` property to the desired break with the required time parameter. The `short_breaks` temporal unit is measured in seconds and the `long_breaks` temporal unit is measured in minutes.
-For example, to extend the break time of `short_break_close_eyes` to 30 seconds and the `long_break_walk` to 5 minutes, modify the configuration file as given below.
+Add the optional `time` property to the desired break with the required time parameter. The time unit is seconds.
+For example, to extend the break time of `short_break_close_eyes` to 30 seconds and the `long_break_walk` to 5 minutes (300 seconds), modify the configuration file as given below.
 
 ```
 ...
@@ -91,7 +91,7 @@ For example, to extend the break time of `short_break_close_eyes` to 30 seconds
 "long_breaks": [
     {
         "name": "long_break_walk",
-        "time": 5
+        "time": 300
     },
     {
         "name": "long_break_lean_back"

safeeyes/safeeyes/SafeEyesCore.py

@@ -78,12 +78,11 @@ def initialize(self, config, language):
 			name = language['exercises'][long_break_config['name']]
 			break_time = long_break_config.get('time', self.long_break_duration)
 			audible_alert = long_break_config.get('audible_alert', config['audible_alert'])
+
 			# Validate time value
 			if not isinstance(break_time, int) or break_time <= 0:
-				logging.error('Invalid time in short break: ' + str(long_break_config))
+				logging.error('Invalid time in long break: ' + str(short_break_config))
 				continue
-			else:
-				break_time = break_time * 60	# Convert to seconds
 
 			self.long_break_exercises.append([name, break_time, audible_alert])
 

safeeyes/safeeyes/config/lang/fr.json

@@ -5,39 +5,39 @@
     },
     "exercises": {
         "short_break_close_eyes": "Fermez bien vos yeux",
-        "short_break_roll_eyes": "Roulez vos yeux",
-        "short_break_rotate_clockwise": "Faites rouler vos yeux dans le sens des aiguilles d'une montre",
-        "short_break_rotate_counter_clockwise": "Faites rouler vos yeux dans le sens contraire des aiguilles d'une montre",
+        "short_break_roll_eyes": "Faites rouler vos yeux",
+        "short_break_rotate_clockwise": "Faites rouler vos yeux dans le sens horaire",
+        "short_break_rotate_counter_clockwise": "Faites rouler vos yeux dans le sens antihoraire",
         "short_break_blink": "Clignez des yeux",
-        "short_break_focus_far_distance": "Focus sur un point dans la distance lointaine",
+        "short_break_focus_far_distance": "Regardez un point au loin",
         "short_break_drink_water": "Buvez de l'eau",
-        "long_break_walk": "Allez marcher un peu",
-        "long_break_lean_back": "Reposez-vous un moment dans votre siège"
+        "long_break_walk": "Marchez un peu",
+        "long_break_lean_back": "Adossez-vous à votre siège et relaxez"
     },
     "messages": {
-        "ready_for_a_break": "Prêt pour une pause dans {} secondes",
-        "disabled_until_restart": "Disabled until restart",
-        "disabled_until_x": "Disabled until {}",
+        "ready_for_a_break": "Préparez-vous à une pause dans {} secondes",
+        "disabled_until_restart": "Désactivé jusqu'au redémarrage",
+        "disabled_until_x": "Désactivé jusqu'à {}",
         "next_break_at": "Prochaine pause à {}"
     },
     "ui_controls": {
-        "skip": "Annuler",
+        "skip": "Ignorer",
         "short_break_duration": "Durée d'une pause courte (en secondes)",
         "long_break_duration": "Durée d'une pause longue (en secondes)",
-        "interval_between_two_breaks": "Intervalle entre deux pauses",
+        "interval_between_two_breaks": "Intervalle entre deux pauses (en minutes)",
         "no_of_short_breaks_between_two_long_breaks": "Nombre de pauses courtes entre deux pauses longues",
-        "time_to_prepare_for_break": "Temps pour se préparer avant une pause (en secondes)",
-        "idle_time": "Minimum idle time to pause (in minutes)",
-        "strict_break": "Pause non annulable (Cacher le bouton Annuler)",
-        "audible_alert": "Audible alert at the end of break",
-        "language": "La language",
+        "time_to_prepare_for_break": "Temps de préparation à une pause (en secondes)",
+        "idle_time": "Durée minimale d'une pause (en minutes)",
+        "strict_break": "Pause stricte (cacher le bouton Ignorer)",
+        "audible_alert": "Alerte sonore à la fin d'une pause",
+        "language": "Langue",
         "enable": "Activer Safe Eyes",
-        "disable": "Disable Safe Eyes",
-        "for_x_minutes": "For {} Minutes",
-        "for_x_hour": "For {} Hour",
-        "for_x_hours": "For {} Hours",
-        "until_restart": "Until restart",
-        "settings": "Réglages",
+        "disable": "Désactiver Safe Eyes",
+        "for_x_minutes": "Pendant {} minutes",
+        "for_x_hour": "Pendant {} heure",
+        "for_x_hours": "Pendant {} heures",
+        "until_restart": "Jusqu'au redémarrage",
+        "settings": "Paramètres",
         "about": "À propos",
         "quit": "Quitter",
         "save": "Enregistrer",