864254 - disable some more options in preferences dialog according to the state of options they are dependent on

Status: RESOLVED FIXED

(Thunderbird :: Preferences, defect)

Description

[:aceman]

I have identified that these dependent disabling can be done:
Composition -> Addressing:
- disable address book dropdown when "Automatically add outgoing e-mail addresses to my:" is unchecked.

Composition -> Spelling:
- disable language dropdown when both spelling options are unchecked.

General:
- disable "Location" textbox when "When TB launches, show start page in the message area" is unchecked.
- disable the "Customize" button, when "Show an alert" is unchecked.

Display -> Formatting -> Colors...:
- disable "text" and "background" color pickers if "Use system colors" is checked. Are these really exclusive?

Attachments -> Outgoing:
- disable the numeric widget when "Offer to share for file larger than" is unchecked.
Also I am not sure if the "share" word is appropriate here as we want to only show the attachment to the recipient not any more public type of sharing.

Any ideas why on of these should not be done?

Comment 1

[:aceman]

Should have been:
Any ideas why ANY of these should not be done?

Comment 2

[Magnus Melin [:mkmelin]]

Re language: i suppose that's still the default language for manual spell-check invocation.

Comment 3

[:aceman]

Thanks. Maybe we could mention that in the panel?

Comment 4

[:aceman]

Attachment: patch

So this seems almost done. There is one problem: there seems to be no style for a disabled <colorpicker> . I have not yet got a reply from toolkit guys.

Comment 5

[rsx11m]

There are some Toolkit definitions for disabled colorpickers for in-content UI,
> http://mxr.mozilla.org/comm-central/search?string=colorpicker&find=inContentUI.css

Mostly just graying the text color and applying some transparency. I'm wondering why such definitions weren't made in the global/colorpicker.css files.

Comment 6

[Richard Marti (:Paenglab)]

(In reply to rsx11m from comment #5)
> There are some Toolkit definitions for disabled colorpickers for in-content
> UI,
> > http://mxr.mozilla.org/comm-central/search?string=colorpicker&find=inContentUI.css
> 
> Mostly just graying the text color and applying some transparency. I'm
> wondering why such definitions weren't made in the global/colorpicker.css
> files.

The buttons have already -moz-appearance: button; and disable="true" let them appear disabled. What's needed is something like:

colorpicker[type="button"][disbled="true"] > .colorpicker-button-colorbox {
  opacity: 0.6;
}

A new bug for this may be worth it.

Comment 7

[:aceman]

Comment on attachment 741055 [details] [diff] [review]
patch

OK, so let's see if the disabling code is actually accepted and then we can file the theme change.

Comment 8

[rsx11m]

> Display -> Formatting -> Colors...:
> - disable "text" and "background" color pickers if "Use system colors" is checked.

Is it possible for the system color lookup to fail? If yes, the colorpickers would determine the fallback colors. If some system color is always defined and a fallback never occurs, disabling the colorpickers is the right thing to do.

Comment 9

[:aceman]

Good point. The code is in /mozilla and I could not see easily if it can fail. 
It seems Firefox has the same dialog and also does not have the disabling.
So I will remove it.

Comment 10

[:aceman]

Attachment: patch v2

Comment 11

[Magnus Melin [:mkmelin]]

Comment on attachment 742516 [details] [diff] [review]
patch v2

Review of attachment 742516 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM! r=mkmelin

Comment 12

[Blake Winton (:bwinton) (:☕️)]

Comment on attachment 742516 [details] [diff] [review]
patch v2

I agree with Magnus.  ui-r=me.  :)

Comment 13

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/comm-central/rev/99d7494dcc96

Comment 14

[Javi Rueda :javirid]

While working on bug 601263, I was looking at the Error Console. When General pane is opened -every time, in fact- an error is logged into it.

Code in function updateCustomizeAlert() in general.js is refering to the customizeMailAlert button which in Mac OS X is not present. There is a #ifdef: http://mxr.mozilla.org/comm-central/source/mail/components/preferences/general.xul#69.

Comment 15

[:aceman]

This bug is already fixed. Please file your problem as a new bug and I'll look into it. Please CC me on the bug, thanks.