Closed Bug 528793 Opened 15 years ago Closed 11 years ago

"Choose color for text" / "Choose color for background" not disabled like everything else

Categories

(Thunderbird :: Message Compose Window, defect)

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 21.0

People

(Reporter: anon.e.mouse.bug, Assigned: aceman)

References

(Blocks 1 open bug)

Details

(Keywords: polish)

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.5) Gecko/20091105 Fedora/3.5.5-1.fc11 Firefox/3.5.5
Build Identifier: Thunderbird/3.0b4

In the formatting bar, there are buttons which have tooltips reading "Choose color for text" and "Choose color for background" -- when the user selects/focuses a field other than the e-mail body, these buttons are NOT disabled, whereas every other button in the formatting bar is.

Bug is applicable to Thunderbird 2 as well as Thunderbird 3 beta 4.

Reproducible: Always

Steps to Reproduce:
1.Click "Write" button from main Thunderbird window
2.Move cursor back and forth between Subject and Body.
3.Observe that all options but those mentioned are disabled when the cursor is in the Subject input.
Actual Results:  
"Choose color for text" and "Choose color for background" are not disabled.

Expected Results:  
"Choose color for text" and "Choose color for background" should be disabled.
Confirming.
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Linux → All
Hardware: x86 → All
I've added the disabled style for this toolbar item in bug 271730 so let's use it here too.
Assignee: nobody → acelists
Depends on: 271730
Keywords: polish
Version: unspecified → Trunk
Attached patch patchSplinter Review
Attachment #690100 - Flags: ui-review?(bwinton)
Attachment #690100 - Flags: review?(mkmelin+mozilla)
Status: NEW → ASSIGNED
Comment on attachment 690100 [details] [diff] [review]
patch

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

Nice polish! r=mkmelin
Attachment #690100 - Flags: review?(mkmelin+mozilla) → review+
Blocks: TB2SM
Comment on attachment 690100 [details] [diff] [review]
patch

I agree, nice polish!

Thanks, and I apologize for the delay.
Blake.
Attachment #690100 - Flags: ui-review?(bwinton) → ui-review+
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/95e659704cba
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 21.0
Depends on: 883540
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: