<ol> "A-Z" fix

Darin Adler darin at apple.com
Mon Dec 1 21:51:44 CET 2003


After X we would go on to A' instead of making it all the way to Z 
first.

-------------- next part --------------
Index: ChangeLog
===================================================================
RCS file: /local/home/cvs/Labyrinth/WebCore/ChangeLog,v
retrieving revision 1.2315
diff -p -u -u -p -r1.2315 ChangeLog
--- ChangeLog	2003/11/22 00:27:21	1.2315
+++ ChangeLog	2003/12/01 19:18:43
@@ -1,3 +1,11 @@
+2003-12-01  Darin Adler  <darin at apple.com>
+
+        Reviewed by Dave.
+
+        - fixed 3493939: ordered lists with type="A" roll over to A' after X rather than after Z
+
+        * khtml/rendering/render_list.cpp: (toLetter): Change 24 to 26.
+
 === Safari-115 ===
 
 2003-11-21  Maciej Stachowiak  <mjs at apple.com>
@@ -56,7 +64,7 @@
 
 2003-11-21  Chris Blumenberg  <cblu at apple.com>
 
-	Fixed: <rdar://problem/3491907>: When viewing applet, view hierarchy was an empty KWQView
+	Fixed: <rdar://problem/3491907>: When viewing applet, view hierarchy was an empty KWQView
 
         Reviewed by rjw.
 
Index: khtml/rendering/render_list.cpp
===================================================================
RCS file: /local/home/cvs/Labyrinth/WebCore/khtml/rendering/render_list.cpp,v
retrieving revision 1.49
diff -p -u -u -p -r1.49 khtml/rendering/render_list.cpp
--- khtml/rendering/render_list.cpp	2003/10/23 22:23:25	1.49
+++ khtml/rendering/render_list.cpp	2003/12/01 19:18:43
@@ -74,10 +74,10 @@ static QString toRoman( int number, bool
 
 static QString toLetter( int number, int base ) {
     number--;
-    QString letter = (QChar) (base + (number % 24));
-    // Add a "'" at the end of the alphabet
-    for (int i = 0; i < (number / 24); i++) {
-       letter += QString::fromLatin1("'");
+    QString letter = (QChar) (base + (number % 26));
+    // Add a single quote at the end of the alphabet.
+    for (int i = 0; i < (number / 26); i++) {
+       letter += '\'';
     }
     return letter;
 }
-------------- next part --------------


     -- Darin


More information about the Khtml-devel mailing list