Reviewed by Hyatt
authorkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 24 Jan 2005 22:20:02 +0000 (22:20 +0000)
committerkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 24 Jan 2005 22:20:02 +0000 (22:20 +0000)
        Fix for this bug:

        <rdar://problem/3963560> 8A354: Inserting Japanese text to the line head causes line break

        * khtml/editing/htmlediting.cpp:
        (khtml::ReplaceSelectionCommand::doApply): Add one more case where we merge content into the
        existing line. This covers the situation see using the steps to reproduce this bug.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@8440 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog-2005-08-23
WebCore/khtml/editing/htmlediting.cpp

index b5d2aa9..92d437a 100644 (file)
@@ -1,3 +1,15 @@
+2005-01-24  Ken Kocienda  <kocienda@apple.com>
+
+        Reviewed by Hyatt
+
+        Fix for this bug:
+        
+        <rdar://problem/3963560> 8A354: Inserting Japanese text to the line head causes line break
+
+        * khtml/editing/htmlediting.cpp:
+        (khtml::ReplaceSelectionCommand::doApply): Add one more case where we merge content into the
+        existing line. This covers the situation see using the steps to reproduce this bug.
+
 2005-01-21  Maciej Stachowiak  <mjs@apple.com>
 
         Reviewed by Darin.
index aecad49..493f85a 100644 (file)
@@ -3921,7 +3921,7 @@ void ReplaceSelectionCommand::doApply()
         mergeStart = mergeEnd = false;
     }
     else {
-        mergeStart = !isStartOfParagraph(visibleStart);
+        mergeStart = !isStartOfParagraph(visibleStart) || (!m_fragment.hasInterchangeNewline() && !m_fragment.hasMoreThanOneBlock());
         mergeEnd = !m_fragment.hasInterchangeNewline() && m_fragment.hasMoreThanOneBlock() && !isEndOfParagraph(visibleEnd);
     }