From 0d3188365aee89c7510d92fae7617d89a8cddfd0 Mon Sep 17 00:00:00 2001 From: Jordan Date: Tue, 17 Jan 2012 11:42:33 -0500 Subject: [PATCH 1/2] Bugfix to noColors view option --- static/js/pad_editor.js | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/static/js/pad_editor.js b/static/js/pad_editor.js index d336aa874..e838ae040 100644 --- a/static/js/pad_editor.js +++ b/static/js/pad_editor.js @@ -75,8 +75,6 @@ var padeditor = (function() { pad.changeViewOption('useMonospaceFont', $("#viewfontmenu").val() == 'monospace'); }); - - settings.noColors = !settings.noColors; // Inversed so we can pass it to showauthorcolors }, setViewOptions: function(newOptions) { @@ -88,7 +86,7 @@ var padeditor = (function() return defaultValue; } - self.ace.setProperty("showsauthorcolors", settings.noColors); + self.ace.setProperty("showsauthorcolors", !settings.noColors); self.ace.setProperty("rtlIsTrue", settings.rtlIsTrue); From da23a62c49111910070b417d69ca6b6f3bd9642e Mon Sep 17 00:00:00 2001 From: Jordan Date: Thu, 9 Feb 2012 21:55:47 -0500 Subject: [PATCH 2/2] Remove an unused collab client message --- static/js/pad.js | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/static/js/pad.js b/static/js/pad.js index 17c2c800a..9dab9c618 100644 --- a/static/js/pad.js +++ b/static/js/pad.js @@ -570,16 +570,6 @@ var pad = { }; options.view[key] = value; pad.handleOptionsChange(options); - // if the request isn't to hide line numbers then broadcast this to other users - if (key != "showLineNumbers" && key != "useMonospaceFont") - { - pad.collabClient.sendClientMessage( - { - type: 'padoptions', - options: options, - changedBy: pad.myUserInfo.name || "unnamed" - }); - } }, handleOptionsChange: function(opts) {