Ticket #79 (assigned defect)

Opened 5 years ago

Last modified 2 years ago

line breaks don't work properly

Reported by: marcvangend Owned by: mr_lundis
Priority: minor Milestone: 1.x
Component: editor Version: trunk
Keywords: br line break Cc:

Description

When I try to enter a line break <br /> by pressing shift-enter, the cursor moves a little to the right. There is no new line created.

When I enter a space after that, the new line is created, containing the space. Pressing backspace remove the space and the new line.

Next, when I press shift-enter again, the new line comes back immediately. However the padding between the current paragraph and the next paragraph disappears.

(tested in IE 6.0.2800)

Change History

comment:1 Changed 4 years ago by jf.hovinne

  • Status changed from new to assigned
  • Milestone set to 0.5

Confirmed - tricky IE designMode issue. Needs more review.

comment:2 follow-up: ↓ 3 Changed 3 years ago by dor

It seems that the bug occurs in every element that has a CSS background declaration, of any kind. Which means that even the following background declaration:

elem { background:none; }

triggers the bug in IE6.

If the background declaration is removed from an element, the bug wouldn't be triggered.

If you make IE6 to treat the element is if it "hasLayout" (via CSS), then the bug wouldn't be triggered despite any background declaration.

I tried to find a solution but couldn't find.

comment:3 in reply to: ↑ 2 Changed 3 years ago by dor

Replying to dor:

If you make IE6 to treat the element is if it "hasLayout"

instead: "element is if it" it should be: "element as if it"

sorry I can't edit...

comment:4 Changed 3 years ago by donrl

Dor's comment about CSS backgrounds is true in IE7 and IE8 as well.

comment:5 Changed 2 years ago by mr_lundis

I've been looking into this issue. I tried giving the elements layout, but this causes nasty resize-handles to show, at least in IE 8.

This is really a hard issue to solve - considering to leave things as-is for now...

comment:6 Changed 2 years ago by mr_lundis

  • Owner changed from jf.hovinne to mr_lundis
  • Status changed from assigned to new

Okay, I think I might have found a solution (finally!); the oncontrolselect event. Preventing the default action should stop the handles from showing when the element has layout. This also means that it won't be necessary to run the iframe in compatibility mode in IE8.

I haven't tested it yet though.

 http://msdn.microsoft.com/en-us/library/ms537844(VS.85).aspx

comment:7 Changed 2 years ago by mr_lundis

  • Status changed from new to assigned

comment:8 Changed 2 years ago by mr_lundis

  • Priority changed from major to minor
  • Milestone changed from 0.5 to 0.5.x

Preventing the default action on the oncontrolselect event does indeed stop the controls from showing, but it also prevents editing the element.

This is not a blocking issue, and there will be no fix for this in 0.5 I'm afraid. If someone comes up with a fix it'll be included in a future maintenance release.

Note: See TracTickets for help on using tickets.