Reviewed by Ken.
authordarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Jul 2004 20:18:05 +0000 (20:18 +0000)
committerdarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Jul 2004 20:18:05 +0000 (20:18 +0000)
        - fill in some unimplemented methods

        * WebView.subproj/WebHTMLView.m:
        (-[WebHTMLView moveToBeginningOfDocument:]): Implement, using new document granularity.
        (-[WebHTMLView moveToBeginningOfDocumentAndModifySelection:]): Ditto.
        (-[WebHTMLView moveToBeginningOfParagraph:]): Implement, even though the underlying WebCore
        code is not yet implemented.
        (-[WebHTMLView moveToBeginningOfParagraphAndModifySelection:]): Ditto.
        (-[WebHTMLView moveToEndOfDocument:]): More of the same.
        (-[WebHTMLView moveToEndOfDocumentAndModifySelection:]): Ditto.
        (-[WebHTMLView moveToEndOfParagraph:]): More of the same.
        (-[WebHTMLView moveToEndOfParagraphAndModifySelection:]): Ditto.
        (-[WebHTMLView pageDown:]): Added comment describing desired behavior.
        (-[WebHTMLView pageUp:]): Ditto.
        (-[WebHTMLView copyFont:]): Ditto.
        (-[WebHTMLView pasteFont:]): Ditto.
        (-[WebHTMLView changeAttributes:]): Ditto.
        (-[WebHTMLView indent:]): Ditto.

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

WebKit/ChangeLog
WebKit/WebView.subproj/WebHTMLView.m

index f14aa0410ac507854f7eddcd2c8113016edf784a..79b15409663383add54f288057944f6114370ad0 100644 (file)
@@ -1,3 +1,26 @@
+2004-07-29  Darin Adler  <darin@apple.com>
+
+        Reviewed by Ken.
+
+        - fill in some unimplemented methods
+
+        * WebView.subproj/WebHTMLView.m:
+        (-[WebHTMLView moveToBeginningOfDocument:]): Implement, using new document granularity.
+        (-[WebHTMLView moveToBeginningOfDocumentAndModifySelection:]): Ditto.
+        (-[WebHTMLView moveToBeginningOfParagraph:]): Implement, even though the underlying WebCore
+        code is not yet implemented.
+        (-[WebHTMLView moveToBeginningOfParagraphAndModifySelection:]): Ditto.
+        (-[WebHTMLView moveToEndOfDocument:]): More of the same.
+        (-[WebHTMLView moveToEndOfDocumentAndModifySelection:]): Ditto.
+        (-[WebHTMLView moveToEndOfParagraph:]): More of the same.
+        (-[WebHTMLView moveToEndOfParagraphAndModifySelection:]): Ditto.
+        (-[WebHTMLView pageDown:]): Added comment describing desired behavior.
+        (-[WebHTMLView pageUp:]): Ditto.
+        (-[WebHTMLView copyFont:]): Ditto.
+        (-[WebHTMLView pasteFont:]): Ditto.
+        (-[WebHTMLView changeAttributes:]): Ditto.
+        (-[WebHTMLView indent:]): Ditto.
+
 2004-07-28  Chris Blumenberg  <cblu@apple.com>
 
        Fixed: <rdar://problem/3737864> Can not download image from web by using drag and drop
index e8c00352479a963bb230bfdc0571557066f50bf7..2cc0e9a2c990282911f2ffe0f2c8e5fec08b2cd2 100644 (file)
@@ -2381,12 +2381,12 @@ static WebHTMLView *lastHitView = nil;
 
 - (void)moveToBeginningOfDocument:(id)sender
 {
-    ERROR("unimplemented");
+    [self _alterCurrentSelection:WebSelectByMoving direction:WebSelectLeft granularity:WebSelectByDocument];
 }
 
 - (void)moveToBeginningOfDocumentAndModifySelection:(id)sender
 {
-    ERROR("unimplemented");
+    [self _alterCurrentSelection:WebSelectByExtending direction:WebSelectLeft granularity:WebSelectByDocument];
 }
 
 - (void)moveToBeginningOfLine:(id)sender
@@ -2401,22 +2401,22 @@ static WebHTMLView *lastHitView = nil;
 
 - (void)moveToBeginningOfParagraph:(id)sender
 {
-    ERROR("unimplemented");
+    [self _alterCurrentSelection:WebSelectByMoving direction:WebSelectLeft granularity:WebSelectByParagraph];
 }
 
 - (void)moveToBeginningOfParagraphAndModifySelection:(id)sender
 {
-    ERROR("unimplemented");
+    [self _alterCurrentSelection:WebSelectByExtending direction:WebSelectLeft granularity:WebSelectByParagraph];
 }
 
 - (void)moveToEndOfDocument:(id)sender
 {
-    ERROR("unimplemented");
+    [self _alterCurrentSelection:WebSelectByMoving direction:WebSelectRight granularity:WebSelectByDocument];
 }
 
 - (void)moveToEndOfDocumentAndModifySelection:(id)sender
 {
-    ERROR("unimplemented");
+    [self _alterCurrentSelection:WebSelectByExtending direction:WebSelectRight granularity:WebSelectByDocument];
 }
 
 - (void)moveToEndOfLine:(id)sender
@@ -2431,12 +2431,12 @@ static WebHTMLView *lastHitView = nil;
 
 - (void)moveToEndOfParagraph:(id)sender
 {
-    ERROR("unimplemented");
+    [self _alterCurrentSelection:WebSelectByMoving direction:WebSelectRight granularity:WebSelectByParagraph];
 }
 
 - (void)moveToEndOfParagraphAndModifySelection:(id)sender
 {
-    ERROR("unimplemented");
+    [self _alterCurrentSelection:WebSelectByExtending direction:WebSelectRight granularity:WebSelectByParagraph];
 }
 
 - (void)moveUp:(id)sender
@@ -2491,11 +2491,13 @@ static WebHTMLView *lastHitView = nil;
 
 - (void)pageDown:(id)sender
 {
+    // FIXME: This should to scroll page down, then move the caret to the top left.
     ERROR("unimplemented");
 }
 
 - (void)pageUp:(id)sender
 {
+    // FIXME: This should to scroll page up, then move the caret to the top left.
     ERROR("unimplemented");
 }
 
@@ -2573,11 +2575,16 @@ static WebHTMLView *lastHitView = nil;
 
 - (void)copyFont:(id)sender
 {
+    // Font is RTF with a single character in it.
+    // NSTextView uses the first character in the selection, or a space if there are no characters.
     ERROR("unimplemented");
 }
 
 - (void)pasteFont:(id)sender
 {
+    // FIXME: Support RTF to HTML (or DOM) conversion.
+    // Font is RTF. Call fontAttributesInRange to extract the attributes to apply.
+    // Then convert the RTF into CSS.
     ERROR("unimplemented");
 }
 
@@ -2653,6 +2660,8 @@ static WebHTMLView *lastHitView = nil;
 
 - (void)changeAttributes:(id)sender
 {
+    // Used for various fancy stuff from the font panel.
+    // Turn the attributes into CSS and then call applyStyle.
     ERROR("unimplemented");
 }
 
@@ -2746,6 +2755,8 @@ static WebHTMLView *lastHitView = nil;
 
 - (void)indent:(id)sender
 {
+    // Figure out current indent level.
+    // Turn new indent level into CSS, then call applyStyle:.
     ERROR("unimplemented");
 }