fixed Theme Tweaker in Epyrus

[sofrosune]

Hi!

I post this here because the Theme Tweaker extension is the same one in both Pale Moon and Epyrus (same release date and all). This could also belong to the Epyrus board, nevertheless.

So, yesterday, I installed this extension and it was not working at all. I tried to solve this by checking the output from the Developer Toolbox and trying to find the source of the problem, to see what should I modify inside the .xpi file. I made it work by adding Epyrus "recognition" where it was relevant, and here is the diff of what I added:

Code: Select all

diff --git a/bootstrap.js b/bootstrap.js
index 168fa69..413cbf2 100644
--- a/bootstrap.js
+++ b/bootstrap.js
@@ -12,6 +12,7 @@ var ThemeFontSizeChanger = {
 	FIREFOX_ID:"{ec8030f7-c20a-464f-9b0e-13a3a9e97384}", // Firefox/Basilisk/Waterfox
 	SEAMONKEY_ID:"{92650c4d-4b8e-4d2a-b7eb-24ecf4f6b63a}", // Seamonkey
 	PM_ID:"{8de7fcbb-c55c-4fbe-bfc5-fc555c87dbc4}", // Pale Moon
+	EPYRUS_ID:"{29877c1d-27df-4421-9a79-382c31470151}", // Epyrus
 	appInfo:Components.classes["@mozilla.org/xre/app-info;1"].getService(Components.interfaces.nsIXULAppInfo),
 	addonLiteralName:"Theme Tweaker", // Keep in sync with em:name in install.rdf
   addonShortName:"TTweak", // Short name used on toolbarbutton label (when enabled)
@@ -203,6 +204,7 @@ var ThemeFontSizeChanger = {
 	openURLInTab:function(window,url,prinstallrequested) {
 		switch (ThemeFontSizeChanger.appInfo.ID) {
 			case ThemeFontSizeChanger.THUNDERBIRD_ID:
+			case ThemeFontSizeChanger.EPYRUS_ID:
 				// Thunderbird's "openTab" implementation for the "contentTab" mode
 				// automatically switches to an existing tab containing the URL we are
 				// opening, so we don't have to check for one here.
@@ -1068,6 +1070,7 @@ var ThemeFontSizeChangerBootstrapAddon = {
 	getWindowType:function(){
 		switch (ThemeFontSizeChanger.appInfo.ID) {
 			case ThemeFontSizeChanger.THUNDERBIRD_ID:
+			case ThemeFontSizeChanger.EPYRUS_ID:
 				return "mail:3pane";	
 				break;
 			default:{
@@ -1370,4 +1373,4 @@ function shutdown(data, reason) {
 	} catch(e){
 		ThemeFontSizeChangerBootstrapAddon.lg(e,1);
 	}
-}
\ No newline at end of file
+}
diff --git a/chrome/content/themefontsizechanger.js b/chrome/content/themefontsizechanger.js
index 06ea23d..0957cb2 100644
--- a/chrome/content/themefontsizechanger.js
+++ b/chrome/content/themefontsizechanger.js
@@ -5,6 +5,7 @@
 		THUNDERBIRD_ID:"{3550f703-e582-4d05-9a08-453d09bdfdc6}", // Thunderbird
 		FIREFOX_ID:"{ec8030f7-c20a-464f-9b0e-13a3a9e97384}", // Firefox/Basilisk/Waterfox
 		PALEMOON_ID:"{8de7fcbb-c55c-4fbe-bfc5-fc555c87dbc4}", // Pale Moon
+		EPYRUS_ID:"{29877c1d-27df-4421-9a79-382c31470151}", // Epyrus
     addonLiteralName:"Theme Tweaker", // Keep in sync with em:name in install.rdf
     addonShortName:"TTweak", // Short name used on toolbarbutton label (when enabled)
 		appInfo:Components.classes["@mozilla.org/xre/app-info;1"].getService(Components.interfaces.nsIXULAppInfo),		
diff --git a/install.rdf b/install.rdf
index 44be9a6..3f9239d 100644
--- a/install.rdf
+++ b/install.rdf
@@ -56,5 +56,12 @@
 				<em:maxVersion>27.*</em:maxVersion>
 			</Description>
 		</em:targetApplication>
+		<em:targetApplication>
+			<Description>
+				<em:id>{29877c1d-27df-4421-9a79-382c31470151}</em:id><!-- epyrus -->
+				<em:minVersion>1.0.0</em:minVersion>
+				<em:maxVersion>2.9.*</em:maxVersion>
+			</Description>
+		</em:targetApplication>
 	</Description>
-</RDF>
\ No newline at end of file
+</RDF>

I attach the working extension for the interested users.

[sofrosune]

For some reason I forgot that extensions could have source repositories when I made this thread .

I did a git pull request and it got merged by Lootyhoof . So it's better to not install (or uninstall) the xpi that I attached.