Editor best be done when NotifyDocumentCreated is called, or else I need a NotifyDocumentReallyDone

ASSIGNED
Assigned to

Status

()

Core
Editor
ASSIGNED
17 years ago
9 years ago

People

(Reporter: Joseph Elwell, Assigned: Charles Manske)

Tracking

Trunk
mozilla1.2beta
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: DEPENDS, URL)

(Reporter)

Description

17 years ago
So I want to set the default text styles in my editorshell based off some prefs.
The correct way seems to be call editorShell's RegisterDocumentStateListener and
in NotifyDocumentCreated I should call editorShell.SetTextProperty("i", "", "");
and other SetTextProperty calls.

The only problem:
When NotifyDocumentCreated is called and I set the text properties, moments
later my properties are all forgotten. Simon says it's likely that some
stylesheets are loaded into the editor after the NotifyDocumentCreated call is
made.
(Assignee)

Comment 1

17 years ago
Yes, we are loading style sheets that are required for editing once the page
loading is completed, but NotifyDocumentCreated is called first.
Interesting problem
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.8
(Assignee)

Comment 2

17 years ago
moving milestone
Target Milestone: mozilla0.9.8 → mozilla0.9.9
(Assignee)

Comment 3

17 years ago
This problem will go away after I move the style sheet loading into the JS code
as part of bug 124265; I will have more control over the order of things hopefully
the editor's NotifyDocumentCreated listener will finish before any that others
add. If that's not true, I'll supply a method for you to call to be sure style
sheets are loaded first.
Depends on: 124265
Keywords: nsbeta1+
(Assignee)

Updated

17 years ago
Whiteboard: DEPENDS
(Assignee)

Comment 4

17 years ago
Unfortunately, this is going to require rewriting nsIEditorStyleSheets interface.
Changing milestone.
Target Milestone: mozilla0.9.9 → mozilla1.0

Comment 5

16 years ago
Please update this bug with an [adt1] - [adt3] impact rating (or take it off the
list if it doesn't even rate adt3.)  Thanks!
(Assignee)

Comment 6

16 years ago
No response from jelwell if this is criticaly, so moviving milestone out.
Target Milestone: mozilla1.0 → mozilla1.2beta

Comment 7

16 years ago
marking nsbeta1- per ADT.
Keywords: nsbeta1+ → nsbeta1-
(Assignee)

Updated

16 years ago
Keywords: nsbeta1- → nsbeta1

Comment 8

15 years ago
adt: nsbeta1-
Keywords: nsbeta1 → nsbeta1-
QA Contact: sujay → editor
You need to log in before you can comment on or make changes to this bug.