Composer UI needs to adapt to editing plain text files

VERIFIED FIXED in mozilla0.9.1

Status

()

P3
normal
VERIFIED FIXED
18 years ago
17 years ago

People

(Reporter: sfraser_bugs, Assigned: sfraser_bugs)

Tracking

Trunk
mozilla0.9.1
All
Mac System 8.5
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

18 years ago
Because we can load up text/plain files in composer, our commands should be 
enabled appropriately for plain text editing in the main composer window. If I 
fix bug 63515, this will be especially important.

It stikes me that we already disable a bunch of commands in HTML source mode. We 
should just keep most of the same commands disabled all the time when editing 
text/plain.
(Assignee)

Updated

18 years ago
Blocks: 63515
(Assignee)

Comment 1

18 years ago
Created attachment 34858 [details] [diff] [review]
Command updating for text mode patch
(Assignee)

Comment 2

18 years ago
Attached a patch to disable commands when  not editing HTML (ie. we are either in 
source mode, or editing a text file. This mostly works, but there are a few 
special cases we need to handle. You'll have to apply this over the patch in 
attachment 34682 [details] [diff] [review].

Comment 3

18 years ago
Since Simon did all the work, it only seems fair to reassign to him and let
him checkin and take proper credit!
I'll be happy to review this.
Assignee: cmanske → sfraser

Comment 4

18 years ago
9.1 for the presnet -- Simon, if you are unable to get this in quickly, then 
please move this one out to 1.0
Priority: -- → P3
Target Milestone: --- → mozilla0.9.1

Comment 5

18 years ago
I believe the patch was checked in lastnight. r=brade sr=kin@netscape.com
(Assignee)

Comment 6

18 years ago
Was checked in.
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 7

18 years ago
verified in 6/7 build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.