git://git.webkit.org / WebKit-https.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: 3c9917f)
Reviewed by Hyatt
authorkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 17 Feb 2005 17:21:58 +0000 (17:21 +0000)
committerkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 17 Feb 2005 17:21:58 +0000 (17:21 +0000)
        Fix for this bug:

        The styling code did not move to the next node when the starting position was
        at the last offset of a node. Instead, it styled it. Clearly wrong. Solution
        is to borrow a check from the delete algorithm. Though I call caretMaxOffset(),
        which I consider deprecated, it is still the simplest and most strightforward
        way to ask the right question for this kind of problem.

        * khtml/editing/htmlediting.cpp:
        (khtml::ApplyStyleCommand::applyInlineStyle): Fixed, as described above.
        * layout-tests/editing/style/style-3998892-fix-expected.txt: Added.
        * layout-tests/editing/style/style-3998892-fix.html: Added.

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

LayoutTests/editing/style/style-3998892-fix-expected.txt [new file with mode: 0644] patch | blob
LayoutTests/editing/style/style-3998892-fix.html [new file with mode: 0644] patch | blob
WebCore/ChangeLog-2005-08-23 patch | blob | history
WebCore/khtml/editing/htmlediting.cpp patch | blob | history

diff --git a/LayoutTests/editing/style/style-3998892-fix-expected.txt b/LayoutTests/editing/style/style-3998892-fix-expected.txt
new file mode 100644 (file)
index 0000000..8dc3a00
--- /dev/null
+++ b/LayoutTests/editing/style/style-3998892-fix-expected.txt
@@ -0,0 +1,41 @@
+layer at (0,0) size 800x600
+  RenderCanvas at (0,0) size 800x600
+layer at (0,0) size 800x600
+  RenderBlock {HTML} at (0,0) size 800x600
+    RenderBody {BODY} at (8,8) size 784x584
+      RenderBlock {DIV} at (0,0) size 784x184 [border: (2px solid #0000FF)]
+        RenderBlock {DIV} at (14,14) size 756x84
+          RenderText {TEXT} at (0,0) size 67x28
+            text run at (0,0) width 67: "Tests: "
+          RenderBR {BR} at (0,0) size 0x0
+          RenderText {TEXT} at (0,28) size 113x28
+            text run at (0,28) width 113: "Bug fix for "
+          RenderInline {A} at (0,0) size 260x28 [color=#0000EE]
+            RenderText {TEXT} at (113,28) size 260x28
+              text run at (113,28) width 260: "<rdar://problem/3998892>"
+          RenderText {TEXT} at (373,28) size 698x56
+            text run at (373,28) width 325: " REGRESSION (Mail): bolding a"
+            text run at (0,56) width 93: "selection "
+            text run at (93,56) width 551: "from end of line changes unselected text on starting line."
+        RenderBlock {DIV} at (14,114) size 756x56
+          RenderText {TEXT} at (0,0) size 189x28
+            text run at (0,0) width 189: "Expected Results: "
+          RenderBR {BR} at (0,0) size 0x0
+          RenderText {TEXT} at (0,28) size 604x28
+            text run at (0,28) width 604: "Should see first line (\"foo\") not bold; second line (\"bar\") bold."
+      RenderBlock {DIV} at (0,208) size 784x60
+        RenderBlock {DIV} at (0,0) size 784x60 [border: (2px solid #FF0000)]
+          RenderBlock {DIV} at (2,2) size 780x28
+            RenderText {TEXT} at (0,0) size 32x28
+              text run at (0,0) width 32: "foo"
+          RenderBlock {DIV} at (2,30) size 780x28
+            RenderInline {B} at (0,0) size 36x28
+              RenderText {TEXT} at (0,0) size 36x28
+                text run at (0,0) width 36: "bar"
+selection is RANGE:
+start:      position 3 of child 1 {TEXT} of child 2 {DIV} of child 1 {DIV} of root {DIV}
+upstream:   position 3 of child 1 {TEXT} of child 2 {DIV} of child 1 {DIV} of root {DIV}
+downstream: position 3 of child 1 {TEXT} of child 2 {DIV} of child 1 {DIV} of root {DIV}
+end:        position 3 of child 1 {TEXT} of child 1 {B} of child 4 {DIV} of child 1 {DIV} of root {DIV}
+upstream:   position 3 of child 1 {TEXT} of child 1 {B} of child 4 {DIV} of child 1 {DIV} of root {DIV}
+downstream: position 3 of child 1 {TEXT} of child 1 {B} of child 4 {DIV} of child 1 {DIV} of root {DIV}
diff --git a/LayoutTests/editing/style/style-3998892-fix.html b/LayoutTests/editing/style/style-3998892-fix.html
new file mode 100644 (file)
index 0000000..a27962c
--- /dev/null
+++ b/LayoutTests/editing/style/style-3998892-fix.html
@@ -0,0 +1,61 @@
+<html> 
+<head>
+
+<style>
+.editing { 
+    border: 2px solid red; 
+    font-size: 24px; 
+}
+.explanation { 
+    border: 2px solid blue; 
+    padding: 12px; 
+    font-size: 24px; 
+    margin-bottom: 24px;
+}
+.scenario { margin-bottom: 16px;}
+.scenario:first-line { font-weight: bold; margin-bottom: 16px;}
+.expected-results:first-line { font-weight: bold }
+</style>
+<script src=../editing.js language="JavaScript" type="text/JavaScript" ></script>
+
+<script>
+
+function editingTest() {
+    for (i = 0; i < 3; i++)
+        moveSelectionForwardByCharacterCommand();
+    extendSelectionForwardByLineCommand();
+    boldCommand();
+}
+
+</script>
+
+<title>Editing Test</title> 
+</head> 
+<body>
+
+<div class="explanation">
+<div class="scenario">
+Tests: 
+<br>
+Bug fix for <a href="rdar://problem/3998892">&lt;rdar://problem/3998892&gt;</a> REGRESSION (Mail): bolding a selection 
+from end of line changes unselected text on starting line.
+</div>
+<div class="expected-results">
+Expected Results:
+<br>
+Should see first line ("foo") not bold; second line ("bar") bold. 
+</div>
+</div>
+
+<div contenteditable id="root" style="word-wrap: break-word; -khtml-nbsp-mode: space; -khtml-line-break: after-white-space;">
+<div id="test" class="editing">
+<div>foo</div>
+<div>bar</div>
+</div>
+
+<script>
+runEditingTest();
+</script>
+
+</body>
+</html>
diff --git a/WebCore/ChangeLog-2005-08-23 b/WebCore/ChangeLog-2005-08-23
index e79e8ebb19acacff4edeafb1b2bd045b7b00f976..0f16b248af3e13442f97bdd9805ecc99f310070b 100644 (file)
--- a/WebCore/ChangeLog-2005-08-23
+++ b/WebCore/ChangeLog-2005-08-23
@@ -1,3 +1,20 @@
+2005-02-17  Ken Kocienda  <kocienda@apple.com>
+
+        Reviewed by Hyatt
+
+        Fix for this bug:
+        
+        The styling code did not move to the next node when the starting position was 
+        at the last offset of a node. Instead, it styled it. Clearly wrong. Solution 
+        is to borrow a check from the delete algorithm. Though I call caretMaxOffset(),
+        which I consider deprecated, it is still the simplest and most strightforward
+        way to ask the right question for this kind of problem.
+
+        * khtml/editing/htmlediting.cpp:
+        (khtml::ApplyStyleCommand::applyInlineStyle): Fixed, as described above.
+        * layout-tests/editing/style/style-3998892-fix-expected.txt: Added.
+        * layout-tests/editing/style/style-3998892-fix.html: Added.
+
 2005-02-16  Chris Blumenberg  <cblu@apple.com>
        
        Fixed: <rdar://problem/3954842> Forward/reply to an HTML email can result in nothing (in cases with <link> tags for CSS)
diff --git a/WebCore/khtml/editing/htmlediting.cpp b/WebCore/khtml/editing/htmlediting.cpp
index ca5b817b8311718c4f10742882977080541931b3..a1cf7ec34f8914c6fbc521cb42199a67dc2a223e 100644 (file)
--- a/WebCore/khtml/editing/htmlediting.cpp
+++ b/WebCore/khtml/editing/htmlediting.cpp
@@ -1472,6 +1472,8 @@ void ApplyStyleCommand::applyInlineStyle(CSSMutableStyleDeclarationImpl *style)
     }
     else {
         NodeImpl *node = start.node();
+        if (start.offset() >= start.node()->caretMaxOffset())
+            node = node->traverseNextNode();
         while (1) {
             if (node->childNodeCount() == 0 && node->renderer() && node->renderer()->isInline()) {
                 NodeImpl *runStart = node;
Mirror of the WebKit open source project from svn.webkit.org.