Changeset 673

Timestamp:

06/30/08 23:22:11 (5 months ago)

Author:

kaz

Message:

Fixed #120

• Added 'Show Popup' setting to preference panel
• Changed positive wording in preference panel
• Remove cookie settings. TwitterFox no longer overwrite user cookie

Location:

trunk/twitternotifier

Files:

• components/nsTwitterNotifier.js (modified) (3 diffs)
• content/login.js (modified) (2 diffs)
• content/login.xul (modified) (1 diff)
• locale/ca-AD/twitternotifier.pref.dtd (modified) (1 diff)
• locale/de-DE/twitternotifier.pref.dtd (modified) (1 diff)
• locale/el-GR/twitternotifier.pref.dtd (modified) (1 diff)
• locale/en-US/twitternotifier.pref.dtd (modified) (1 diff)
• locale/es-CL/twitternotifier.pref.dtd (modified) (1 diff)
• locale/es-ES/twitternotifier.pref.dtd (modified) (1 diff)
• locale/fi-FI/twitternotifier.pref.dtd (modified) (1 diff)
• locale/fr-FR/twitternotifier.pref.dtd (modified) (1 diff)
• locale/gl-ES/twitternotifier.pref.dtd (modified) (1 diff)
• locale/it-IT/twitternotifier.pref.dtd (modified) (1 diff)
• locale/ja-JP/twitternotifier.pref.dtd (modified) (1 diff)
• locale/nb-NO/twitternotifier.pref.dtd (modified) (1 diff)
• locale/nl-NL/twitternotifier.pref.dtd (modified) (1 diff)
• locale/pl-PL/twitternotifier.pref.dtd (modified) (1 diff)
• locale/pt-BR/twitternotifier.pref.dtd (modified) (1 diff)
• locale/sr-RS/twitternotifier.pref.dtd (modified) (1 diff)
• locale/sr-YU/twitternotifier.pref.dtd (modified) (1 diff)
• locale/zh-CN/twitternotifier.pref.dtd (modified) (1 diff)
• locale/zh-TW/twitternotifier.pref.dtd (modified) (1 diff)
• tools/migrate.pl (modified) (2 diffs)

trunk/twitternotifier/components/nsTwitterNotifier.js

@@ -110 +110 @@
   },
 
-  asyncOpen: function(noCookie) {
-    this.noCookie = noCookie;
+  asyncOpen: function() {
     this.channel.notificationCallbacks = this;
     this.channel.asyncOpen(this, null);
@@ -189 +188 @@
     //
     if (subject == this.channel) {
-      if (this.noCookie) {
-        if (topic == "http-on-modify-request") {
-          this.httpChannel().setRequestHeader("Cookie", "", false);
-        }
-        else if (topic == "http-on-examine-response") {
-          this.httpChannel().setResponseHeader("Set-Cookie", "", false);
-        }
+      if (topic == "http-on-modify-request") {
+        this.httpChannel().setRequestHeader("Cookie", "", false);
+      }
+      else if (topic == "http-on-examine-response") {
+        this.httpChannel().setResponseHeader("Set-Cookie", "", false);
       }
 
@@ -332 +329 @@
       .getService(Components.interfaces.nsIPrefBranch);
 
-    request.asyncOpen(pref.getBoolPref("extensions.twitternotifier.noCookie"));
+    request.asyncOpen();
     return request;
   },

trunk/twitternotifier/content/login.js

@@ -30 +30 @@
     $("popup-interval").value = popup;
 
-    $("popup-autoclose").checked = !this.util.pref().getBoolPref("autoClose");
-    $("no-cookie").checked = this.util.pref().getBoolPref("noCookie");
+    $("popup-autoclose").checked = this.util.pref().getBoolPref("autoClose");
+    $("balloon-popup").checked   = this.util.pref().getBoolPref("popup");
 
     $("sound").checked         = this.util.pref().getBoolPref("sound");
@@ -284 +284 @@
     this.util.pref().setIntPref("interval", $("refresh-interval").value);
     this.util.pref().setIntPref("popup-interval", $("popup-interval").value);
-    this.util.pref().setBoolPref("autoClose", !$("popup-autoclose").checked);
-    this.util.pref().setBoolPref("noCookie", $("no-cookie").checked);
+    this.util.pref().setBoolPref("autoClose", $("popup-autoclose").checked);
+    this.util.pref().setBoolPref("popup", $("balloon-popup").checked);
 
     this.util.pref().setBoolPref("sound", $("sound").checked);

trunk/twitternotifier/content/login.xul

@@ -124 +124 @@
         <row>
           <hbox align="center">
+            <label value="&twitternotifier.balloon;"/>
+          </hbox>
+          <checkbox id="balloon-popup" label="&twitternotifier.showBalloon;" />
+        </row>
+        <row>
+          <hbox align="center">
             <label value="&twitternotifier.popupBehavior;"/>
           </hbox>
           <checkbox id="popup-autoclose" label="&twitternotifier.popupAutoclose;" />
-        </row>
-        <row>
-          <hbox align="center">
-            <label value="&twitternotifier.cookie;"/>
-          </hbox>
-          <checkbox id="no-cookie" label="&twitternotifier.cookieBehavior;" tooltiptext="&twitternotifier.cookieBehavior.tooltip;"/>
         </row>
         <row>

trunk/twitternotifier/locale/ca-AD/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "Pantalla de la finestra">
 <!ENTITY twitternotifier.popupAutoclose "No tancar finestra automàticament">
-<!ENTITY twitternotifier.cookie "Galeta">
-<!ENTITY twitternotifier.cookieBehavior "No sobreescriure galeta de l&apos;usuari">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "Si actives aquesta casella, TwitterFox usarà les pròpies galetes en comptes de les de l&apos;usuari">
 <!ENTITY twitternotifier.sound "So de notificació">
 <!ENTITY twitternotifier.soundButton "Navegar...">

trunk/twitternotifier/locale/de-DE/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "Fensteranzeige">
 <!ENTITY twitternotifier.popupAutoclose "Fenster nicht automatisch schließen">
-<!ENTITY twitternotifier.cookie "Cookie">
-<!ENTITY twitternotifier.cookieBehavior "Benutzercookie nicht überschreiben">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "Wenn Sie diese Checkbox aktivieren, benutzt TwitterFox ein eigenes statt dem Benutzercookie">
 <!ENTITY twitternotifier.sound "Benachrichtigungssound">
 <!ENTITY twitternotifier.soundButton "Durchsuchen...">

trunk/twitternotifier/locale/el-GR/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "Εμφάνιση παραθύρου">
 <!ENTITY twitternotifier.popupAutoclose "Να μην κλείνει αυτόματα το παράθυρο">
-<!ENTITY twitternotifier.cookie "Cookie">
-<!ENTITY twitternotifier.cookieBehavior "Να μην αντικαθίσταται το cookie του χρήστη">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "Εάν επιλέξετε αυτό το πεδίο, το TwitterFox θα χρησιμοποιεί το δικό του cookie αντί του cookie του χρήστη">
 <!ENTITY twitternotifier.sound "Ήχος ειδοποίησης">
 <!ENTITY twitternotifier.soundButton "Εύρεση...">

trunk/twitternotifier/locale/en-US/twitternotifier.pref.dtd

@@ -21 +21 @@
 <!ENTITY twitternotifier.revertKeyconfig "Revert">
 <!ENTITY twitternotifier.popupBehavior "Window display">
-<!ENTITY twitternotifier.popupAutoclose "Don&apos;t close window automatically">
-<!ENTITY twitternotifier.cookie "Cookie">
-<!ENTITY twitternotifier.cookieBehavior "Don&apos;t overwrite user cookie">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "If you check this box, TwitterFox use own cookie instead of user cookie">
+<!ENTITY twitternotifier.popupAutoclose "Close window automatically">
+<!ENTITY twitternotifier.balloon "Popup display">
+<!ENTITY twitternotifier.showBalloon "Show balloon popup">
 <!ENTITY twitternotifier.sound "Notification sound">
 <!ENTITY twitternotifier.soundButton "Browse...">

trunk/twitternotifier/locale/es-CL/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "Mostrar ventana">
 <!ENTITY twitternotifier.popupAutoclose "No cerrar la ventana automáticamente.">
-<!ENTITY twitternotifier.cookie "Cookie">
-<!ENTITY twitternotifier.cookieBehavior "No sobrescribir cookie de usuario.">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "Si activas esta opción, TwitterFox usará su propia cookie en lugar de la del usuario.">
 <!ENTITY twitternotifier.sound "Sonido de notificación">
 <!ENTITY twitternotifier.soundButton "Navegar...">

trunk/twitternotifier/locale/es-ES/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "Mostrar ventana">
 <!ENTITY twitternotifier.popupAutoclose "No cerrar la ventana automáticamente">
-<!ENTITY twitternotifier.cookie "Cookie">
-<!ENTITY twitternotifier.cookieBehavior "No sobreescribir la cookie de usuario">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "Si activas esta casilla, TwitterFox usará su propia cookie en lugar de la cookie de usuario">
 <!ENTITY twitternotifier.sound "Sonido de aviso">
 <!ENTITY twitternotifier.soundButton "Examinar...">

trunk/twitternotifier/locale/fi-FI/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "Ikkunan esittäminen">
 <!ENTITY twitternotifier.popupAutoclose "Älä sulje ikkunaa automaattisesti">
-<!ENTITY twitternotifier.cookie "Eväste">
-<!ENTITY twitternotifier.cookieBehavior "Älä ylikirjoita käyttäjän evästettä">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "Jos merkitset tämän valintalaatikon, TwitterFox käyttää omaa evästettään käyttäjän evästeen sijaan">
 <!ENTITY twitternotifier.sound "Ilmoitusääni">
 <!ENTITY twitternotifier.soundButton "Selaa...">

trunk/twitternotifier/locale/fr-FR/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "Affichage de la fenêtre">
 <!ENTITY twitternotifier.popupAutoclose "Ne pas fermer la fenêtre automatiquement">
-<!ENTITY twitternotifier.cookie "Cookie">
-<!ENTITY twitternotifier.cookieBehavior "Ne pas remplacer le cookie utilisateur">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "Si vous cochez cette case, TwitterFox utilisera son propre cookie">
 <!ENTITY twitternotifier.sound "Son de notification">
 <!ENTITY twitternotifier.soundButton "Parcourir…">

trunk/twitternotifier/locale/gl-ES/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "Mostrar fiestra">
 <!ENTITY twitternotifier.popupAutoclose "Non pechar a fiestra automaticamente">
-<!ENTITY twitternotifier.cookie "Cookie">
-<!ENTITY twitternotifier.cookieBehavior "Non sobrescribir a cookie do usuario">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "Se marcas este cadro, TwitterFox empregará a sua propia cookie no lugar da cookie de usuario">
 <!ENTITY twitternotifier.sound "Son de notificación">
 <!ENTITY twitternotifier.soundButton "Examinar...">

trunk/twitternotifier/locale/it-IT/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "Visualizza finestra">
 <!ENTITY twitternotifier.popupAutoclose "Non chiuderla automaticamente">
-<!ENTITY twitternotifier.cookie "Cookie">
-<!ENTITY twitternotifier.cookieBehavior "Non sovrascrivere cookie dell&apos;utente">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "Se selezioni questa opzione, TwitterFox userà un proprio cookie invece di quello dell&apos;utente">
 <!ENTITY twitternotifier.sound "Suono di notifica">
 <!ENTITY twitternotifier.soundButton "Seleziona...">

trunk/twitternotifier/locale/ja-JP/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "ウィンドウの表示">
 <!ENTITY twitternotifier.popupAutoclose "ウィンドウを自動的に閉じない">
-<!ENTITY twitternotifier.cookie "クッキー">
-<!ENTITY twitternotifier.cookieBehavior "ユーザーのCookieを上書きしない">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "このボックスをチェックすると、ブラウザとTwitterFoxで異なるアカウントを使うことが出来ます">
+<!ENTITY twitternotifier.balloon "ポップアップの表示">
+<!ENTITY twitternotifier.showBalloon "バルーンポップアップを表示する">
 <!ENTITY twitternotifier.sound "新着を音で通知する">
 <!ENTITY twitternotifier.soundButton "参照...">

trunk/twitternotifier/locale/nb-NO/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "Vindusvisning">
 <!ENTITY twitternotifier.popupAutoclose "Ikke lukk vinduet automatisk">
-<!ENTITY twitternotifier.cookie "Informasjonskapsel">
-<!ENTITY twitternotifier.cookieBehavior "Ikke overskriv brukerens informasjonskapsel">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "Hvis du krysser av her vil TwitterFox bruke sin egen informasjonskapsel">
 <!ENTITY twitternotifier.sound "Varslingslyd">
 <!ENTITY twitternotifier.soundButton "Finn ...">

trunk/twitternotifier/locale/nl-NL/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "Vensterweergave">
 <!ENTITY twitternotifier.popupAutoclose "Venster niet automatisch sluiten">
-<!ENTITY twitternotifier.cookie "Cookie">
-<!ENTITY twitternotifier.cookieBehavior "Gebruikerscookie niet overschrijven">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "Als u dit veld aanvinkt zal TwitterFox een eigen cookie in plaats van een gebruikerscookie gebruiken">
 <!ENTITY twitternotifier.sound "Notificatiegeluid">
 <!ENTITY twitternotifier.soundButton "Bladeren...">

trunk/twitternotifier/locale/pl-PL/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "Wyświetlanie okienka">
 <!ENTITY twitternotifier.popupAutoclose "Nie zamykaj okienka automatycznie">
-<!ENTITY twitternotifier.cookie "Ciasteczko">
-<!ENTITY twitternotifier.cookieBehavior "Nie nadpisuj ciasteczka użytkownika">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "Każ TwitterFoksowi korzystać z własnego ciasteczka zamiast z ciasteczka użytkownika">
 <!ENTITY twitternotifier.sound "Dźwięk powiadomienia">
 <!ENTITY twitternotifier.soundButton "Przeglądaj...">

trunk/twitternotifier/locale/pt-BR/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "Exibição da janela">
 <!ENTITY twitternotifier.popupAutoclose "Não fechar a janela automaticamente">
-<!ENTITY twitternotifier.cookie "Cookie">
-<!ENTITY twitternotifier.cookieBehavior "Não sobrescrever cookie do usuário">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "Se você verificar esta caixa, o TwitterFox usa um cookie próprio em vez do cookie do usuário">
 <!ENTITY twitternotifier.sound "Som de notificação">
 <!ENTITY twitternotifier.soundButton "Escolher arquivo...">

trunk/twitternotifier/locale/sr-RS/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "Приказ прозора">
 <!ENTITY twitternotifier.popupAutoclose "Не затварај прозор аутоматски">
-<!ENTITY twitternotifier.cookie "Колачићи">
-<!ENTITY twitternotifier.cookieBehavior "Не преписуј корисникове колачиће">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "Ако омогућите ову опцију, TwitterFox ће користити своје колачиће уместо корисникових колачића">
 <!ENTITY twitternotifier.sound "Звук за обавештења">
 <!ENTITY twitternotifier.soundButton "Потражи...">

trunk/twitternotifier/locale/sr-YU/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "Prikaz prozora">
 <!ENTITY twitternotifier.popupAutoclose "Ne zatvaraj prozor automatski">
-<!ENTITY twitternotifier.cookie "Kolačići">
-<!ENTITY twitternotifier.cookieBehavior "Ne prepisuj korisnikove kolačiće">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "Ako omogućite ovu opciju, TwitterFox će koristiti svoje kolačiće umesto korisnikovih kolačića">
 <!ENTITY twitternotifier.sound "Zvuk za obaveštenja">
 <!ENTITY twitternotifier.soundButton "Potraži...">

trunk/twitternotifier/locale/zh-CN/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "窗口显示">
 <!ENTITY twitternotifier.popupAutoclose "不要自动关闭窗口">
-<!ENTITY twitternotifier.cookie "Cookie">
-<!ENTITY twitternotifier.cookieBehavior "不要覆写用户Cookie">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "选择此项，TwitterFox会以自己的Cookie取代用户Cookie">
 <!ENTITY twitternotifier.sound "新信息通知声音">
 <!ENTITY twitternotifier.soundButton "浏览...">

trunk/twitternotifier/locale/zh-TW/twitternotifier.pref.dtd

@@ -22 +22 @@
 <!ENTITY twitternotifier.popupBehavior "視窗顯示">
 <!ENTITY twitternotifier.popupAutoclose "不要自動關閉視窗">
-<!ENTITY twitternotifier.cookie "Cookie">
-<!ENTITY twitternotifier.cookieBehavior "不要覆寫使用者的Cookie">
-<!ENTITY twitternotifier.cookieBehavior.tooltip "核取此項，TwitterFox將以自己的Cookie取代使用者的Cookie">
 <!ENTITY twitternotifier.sound "新訊息通知音效">
 <!ENTITY twitternotifier.soundButton "瀏覽...">

trunk/twitternotifier/tools/migrate.pl

@@ -38 +38 @@
 }
 
-my @entities = readentities('en-US', 'twitternotifier.pref.properties');
+my $target = 'twitternotifier.pref.dtd';
+
+my @entities = readentities('en-US', $target);
 
 foreach my $locale (@locales) {
 
-    copy("locale/$locale/twitternotifier.properties", "locale/$locale/twitternotifier.properties.orig");
+    copy("locale/$locale/$target", "locale/$locale/$target.orig");
     
-    open INPUT, "locale/$locale/twitternotifier.properties.orig" or die "Couldn't open file, $!";
-    open OUTPUT, "> locale/$locale/twitternotifier.pref.properties" or die "Couldn't open file, $!";
+    open INPUT, "locale/$locale/$target.orig" or die "Couldn't open file, $!";
+    open OUTPUT, "> locale/$locale/$target" or die "Couldn't open file, $!";
 
     while (<INPUT>) {
         chomp;
         my $line = $_;
-        my @entity = split(/=/, $line, 2);
-        next if scalar @entity != 2;
+
+        my @entity;
+        if ($target =~ /properties$/) {
+            @entity = split(/=/, $line);
+        }
+        else {
+            $line =~ /ENTITY twitternotifier\.(\w+)\s*/;
+            push @entity, $1;
+        }
 
         if (grep(/$entity[0]/, @entities)) {
-            print OUTPUT "$entity[0]=$entity[1]\n";
+            print OUTPUT "$line\n";
         }
     }
@@ -61 +70 @@
 }
 
-my @entities = readentities('en-US', 'twitternotifier.properties');
-
-foreach my $locale (@locales) {
-
-    open INPUT, "locale/$locale/twitternotifier.properties.orig" or die "Couldn't open file, $!";
-    open OUTPUT, "> locale/$locale/twitternotifier.properties" or die "Couldn't open file, $!";
-
-    while (<INPUT>) {
-        chomp;
-        my $line = $_;
-        my @entity = split(/=/, $line, 2);
-        next if scalar @entity != 2;
-
-        if (grep(/$entity[0]/, @entities)) {
-            print OUTPUT "$entity[0]=$entity[1]\n";
-        }
-    }
-    close OUTPUT;
-    close INPUT;
-}