200364 - Using undo produces different visual spacing

Status: RESOLVED WORKSFORME

(Core :: Layout: Block and Inline, defect, P3)

Description

[Kai Engert [:KaiE:]]

I think bug 200360 should be fixed first.
Start with a similar testcase as in that other bug:

open new composer window
input:
  a enter b enter
go before the "a"
use the dropdown to change the type to "paragraph"

(Note that there is no additional spacing between a and b (that is what bug
200360 complains about))

Position the caret in the same line as "a", at the end of the line.
Insert a horizontal line
Hit ctrl-z to undo

Expected behaviour:
  the appearance should be identical with the appearance before
  having inserted the horizontal line

Actual behaviour:
  There is now suddenly a large visual spacing between a and b
  (that spacing is not caused by additional empty line breaks
   between the lines, there now seems to be some margins in effect)


I somehow suspect we might see this fixed as soon as bug 200360 is fixed.
But we should retest once that is done.
Adding dependency.

Comment 1

[Joe Francis]

changin component to layout: block&inline.  The editor is creating correct dom,
it's just not being rendered correctly.

Comment 2

[Kevin McCluskey (gone)]

editorbase+, P3

Comment 3

[Kevin McCluskey (gone)]

-> smontagu

Comment 4

[Stephen Donner [:stephend] Not actively reading bugmail]

Mass-removing myself from cc; search for 12b9dfe4-ece3-40dc-8d23-60e179f64ac1 or any reasonable part thereof, to mass-delete these notifications (and sorry!)

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee didn't login in Bugzilla in the last 7 months.
:jfkthame, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 6

[Jonathan Kew [:jfkthame]]

Trying this in a current Thunderbird message composition window, I can't reproduce the issue described.

Closing as WFM, as this has most likely been resolved as part of layout engine changes over the years. Please re-open with further details and testcase if there is still a problem here.