Closed
Bug 528793
Opened 15 years ago
Closed 12 years ago
"Choose color for text" / "Choose color for background" not disabled like everything else
Categories
(Thunderbird :: Message Compose Window, defect)
Thunderbird
Message Compose Window
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)
4.63 KB,
patch
|
mkmelin
:
review+
bwinton
:
ui-review+
|
Details | Diff | Splinter Review |
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.
Comment 1•15 years ago
|
||
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.
Attachment #690100 -
Flags: ui-review?(bwinton)
Attachment #690100 -
Flags: review?(mkmelin+mozilla)
Comment 4•12 years ago
|
||
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+
Comment 5•12 years ago
|
||
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
Comment 6•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 21.0
You need to log in
before you can comment on or make changes to this bug.
Description
•