Closed Bug 6059 Opened 25 years ago Closed 25 years ago

{yikes} first-letter and editing don't mix

Categories

(Core :: Layout, defect, P2)

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: buster, Assigned: buster)

References

()

Details

In viewer, bring up sample 0. Turn on the editor. Put your cursor between 'm' and 'p' in "Example" of the first line. Hit return. you end up with a display that looks like: "ple 0: Basic HTML Text Styles" in H1 style "ple 0: Basic HTML Text Stylesple 0: Basic HTML Text Styles" in normal style Dumping content, the content looks ok! Maybe a frame bug handling incremental reflow? Then, you have to undo twice to get back to what looks like the beginning state, although the text is screwed up in the display -- "ple 0: Basic HTML Text Styles" repeated 3 times in an H1 style. You can see the two-step undo problem in any simple text document.
Status: NEW → ASSIGNED
accepting bug
Target Milestone: M7
targetting m7. this does indeed look like a layout prob but i need to investigate more.
Assignee: jfrancis → buster
Status: ASSIGNED → NEW
Buster, passing off these two bugs (6059, 6242) to you as previously discussed. These are both candidates for non-editor bugs that could use some triage by a layout savvy person.
Assignee: buster → rickg
Component: Editor → Layout
If you minimize the test case considerably (for convenience) to only the first few elements, and you remove the inline style rules at the top of the document, you will see this problem go away. The problem seems to be with layout's handling of first-letter or first-line. My guess is that it is specifically: :first-letter { color: green; } Sorry I can't get a more complete reduction, I am travelling and I don't have a dev machine loaded with the latest build. Normally, this would be assigned to Kipp, so I'm assigning it to rick. changing component to layout.
Assignee: rickg → kipp
Severity: major → normal
Summary: insert break and undo of insert break are badly broken → {first-letter} insert break and undo of insert break are badly broken
Target Milestone: M7 → M15
Kipp is on sabbatical. Marking bug M15. See also bug 2418.
cc buster, not kostello.
Severity: normal → minor
Status: NEW → ASSIGNED
Priority: P3 → P5
DOM interactions with :first-letter style are known to be *unimplemented*.
Summary: {first-letter} insert break and undo of insert break are badly broken → {first-letter} first-letter and editing don't mix
*** Bug 9444 has been marked as a duplicate of this bug. ***
Updated summary to reflect the real problem.
Marking yikes since its hard to fix.
Severity: minor → major
Priority: P5 → P4
Summary: {first-letter} first-letter and editing don't mix → {yikes} first-letter and editing don't mix
I guess it has nothing to do with editing per se, but is a general problem with incremental reflow and these styles, right? So any DOM script that manipulated the content model when first-letter and/or first-line would be effected in the identical fashion. Maybe that's what the summary should say?
Priority: P4 → P2
Target Milestone: M15 → M10
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
I've updated the frame construction code to allow for append/insert/remove when first-letter is present.
buster, somehow I'm not getting viewer in my daily builds... if you have it handy can you verify this one?
you can verify it in apprunner.
Status: RESOLVED → VERIFIED
verified in 9/15 build...
You need to log in before you can comment on or make changes to this bug.