Florian Quèze [:florian] Assignee
	Description • 12 years ago

The "mail.chat.enabled" pref currently prevents the IM code from being initialized, hides the IM UI from the main window ("Chat" button in the toolbar; menu items) and from the Account Settings dialog (hides IM accounts from the list and hides the "Add Chat account…" menu item), but the "Chat" prefpane is still visible in the pref window.

	Florian Quèze [:florian] Assignee
	Comment 1 • 12 years ago

I wish I could just set .hidden = true on the prefpane, but the preference bindings don't like it that way...

Also, I'm not very sure of what the story is with Services.jsm in the prefwindow with all these dynamically loaded overlay files, so I played safe and used the old Components.classes[...].getService form to get the pref service.

	Blake Winton (:bwinton) (:☕️)
	Comment 2 • 12 years ago

Comment on attachment 608112 [details] [diff] [review]
Patch

>+++ b/mail/components/preferences/jar.mn
>@@ -1,15 +1,17 @@
> messenger.jar:
>+    content/messenger/preferences/preferences.js

So, "preferences.js" seems really generic.  Is there another name we could choose that more accurately describes what it does?
(It probably doesn't matter, so I'm not going to block the review over it, but if you can think of a better name, I'ld appreciate it.  :)

>+    content/messenger/preferences/chat.js

Why are you adding chat.js here?

>+++ b/mail/components/preferences/preferences.js
>@@ -0,0 +1,16 @@
>+/* This Source Code Form is subject to the terms of the Mozilla Public
>+ * License, v. 2.0. If a copy of the MPL was not distributed with this file,
>+ * You can obtain one at http://mozilla.org/MPL/2.0/. */

Love that MPL2!  :)

I mostly like it.  r=me with the chat.js removed, or an explanation of why it's necessary (and wasn't necessary before).

Thanks,
Blake.

	Florian Quèze [:florian] Assignee
	Comment 3 • 12 years ago

(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #2)
> Comment on attachment 608112 [details] [diff] [review]
> Patch
> 
> >+++ b/mail/components/preferences/jar.mn
> >@@ -1,15 +1,17 @@
> > messenger.jar:
> >+    content/messenger/preferences/preferences.js
> 
> So, "preferences.js" seems really generic.  Is there another name we could
> choose that more accurately describes what it does?

For me "preferences.js" in this context means "the JS file loaded with the XUL file named preferences.xul" so I can't think of a better name.
I was quite surprised that this file didn't exist yet.

> >+    content/messenger/preferences/chat.js
> 
> Why are you adding chat.js here?

It's an unrelated change, I pondered filing a separate bug and attaching a separate patch for it, but that seemed like too much clicking.

chat.xul includes chat.js that contains code to disable the subitems of the "Let my contacts know that I'm idle" when that checkbox is unchecked. However that didn't work because I forgot to package chat.js...

	Florian Quèze [:florian] Assignee
	Comment 4 • 12 years ago

http://hg.mozilla.org/comm-central/rev/41aa34975063

	Florian Quèze [:florian] Assignee
	Comment 5 • 12 years ago

Comment on attachment 608112 [details] [diff] [review]
Patch

[Approval Request Comment]
This is required if we want to have the ability to pref off the IM feature.

	Florian Quèze [:florian] Assignee
	Comment 6 • 12 years ago

http://hg.mozilla.org/releases/comm-aurora/rev/7fdb23d4f87b