Closed Bug 26447 Opened 25 years ago Closed 24 years ago

Implement updating of Paragraph and Font face menulists and handle "mixed" selection

Categories

(Core :: DOM: Editor, defect, P1)

Product:
Core
Component:
DOM: Editor
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: mozeditor, Assigned: sfraser_bugs)

References

Details

(Whiteboard: [nsbeta2+][6/01]) 

in composer, the fonts selection popup in the toolbar doesn't do anything.  The 
expected result is that it does do something.  :-)
Joe -- I'm setting this to M14
Target Milestone: M14
actually, it does do something a lot of the time.  But there are times when it 
doesn't.  Changing summary.  Accepting bug.
Status: NEW → ASSIGNED
Summary: FEATURE: font name toolbar item non-functional → FEATURE: font name toolbar item sometimes non-functional
this is not going to happen for m14.
Target Milestone: M14 → M15
moving out to M17
Target Milestone: M15 → M17
Charlie, I need some UI stuff to happen before I can fix this.  When is the code 
that will show a blank font value for mixed font selections going to land?  
Assigning to you so you see this comment.  Assign back to me as needed...
Assignee: jfrancis → cmanske
Status: ASSIGNED → NEW
Toolbars will be finished for beta2 (M16). Doing dialogs first.
Status: NEW → ASSIGNED
Target Milestone: M17 → M16
Changing the summary to better reflect the task.
Summary: FEATURE: font name toolbar item sometimes non-functional → Implement updating of Paragraph and Font face menulists and handle "mixed" selection
This must be done for beta2, I think!
Keywords: nsbeta2
Priority: P3 → P1
Whiteboard: nsbeta2
this won't be resolved by 5/16, but the para and font menus need to be resolved 
before beta 2. The font options are not reflecting the accurate choices, and the 
para format is not correct. We've had complaints regarding both areas. Would 
deeply encourage the PDT team to approve this.
I'm not clear what it is that wont be resolved by 5/16.  I can get the menu 
behavior improved before then.  Are we saying that we wont be able to set them to 
blank values by 5/16?  Charlie?

What I mean is I might not have time to implement setting the state of the 
menulist at all, including the "mixed" state. It's a reasonably simple task,
I'm just trying to concentrate on more difficult problems first.

Joe: Will this block you?
Putting on [nsbeta2+][6/01] radar.  This work must be done by 06/01 or we may 
pull this for PR2.
Whiteboard: nsbeta2 → [nsbeta2+][6/01]
I implemented the method onParagraphFormatChange in editor.js.
The command state notification involves Simon's cool command system, so 
he has volunteered to take if from here.
Assignee: cmanske → sfraser
Status: ASSIGNED → NEW
I have the paragraph updating mostly done.
Status: NEW → ASSIGNED
*** Bug 40201 has been marked as a duplicate of this bug. ***
*** Bug 40202 has been marked as a duplicate of this bug. ***
fixed; thanks to simon and charlie for doing the ui work.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
verified in 5/26 build.
Status: RESOLVED → VERIFIED
Blocks: 635516
You need to log in before you can comment on or make changes to this bug.