Bug 12911 - GoogleDocs: Ordered lists don't update immediately when start attribute changed
Summary: GoogleDocs: Ordered lists don't update immediately when start attribute changed
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P1 Normal
Assignee: Nobody
URL:
Keywords: GoogleBug, InRadar
Depends on:
Blocks: 7712
  Show dependency treegraph
 
Reported: 2007-02-27 09:48 PST by Steven Saviano
Modified: 2007-02-28 08:00 PST (History)
4 users (show)

See Also:


Attachments
Test case for changing the "start" attribute of the ol tag. (431 bytes, text/html)
2007-02-27 09:51 PST, Steven Saviano
no flags Details
testcase (236 bytes, text/html)
2007-02-27 15:01 PST, Justin Garcia
no flags Details
update list markers when start attribute changes (11.35 KB, patch)
2007-02-27 18:24 PST, Antti Koivisto
mjs: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Steven Saviano 2007-02-27 09:48:29 PST
Steps:
1) Make an ordered list. (using <ol>)
2) Try to change the "start" attribute of the tag

Expected Result:
1) The underlying html changes and the list redraws to display the change to "start".

Actual Result:
1) The underlying html changes BUT the list does not redraw to display the cahnge to "start".

I tested this on r19879.
Comment 1 Steven Saviano 2007-02-27 09:51:51 PST
Created attachment 13403 [details]
Test case for changing the "start" attribute of the ol tag.
Comment 2 David Harrison 2007-02-27 10:50:50 PST
rdar://5026069
Comment 3 Justin Garcia 2007-02-27 12:46:48 PST
The list isn't updated to reflect changes made to the list marker appearance, either.  Re-titling.
Comment 4 Justin Garcia 2007-02-27 13:17:24 PST
The property changes are eventually visible, but it takes 5-10 seconds.  Looks like they update when GoogleDocs autosaves?
Comment 5 Justin Garcia 2007-02-27 15:01:34 PST
Created attachment 13411 [details]
testcase

Automated testcase that changes both the start attribute and the list-style-type css property.  Neither change has an effect on the list visually.
Comment 6 mitz 2007-02-27 15:08:18 PST
(In reply to comment #5)
> Created an attachment (id=13411) [edit]
> testcase
> 
> Automated testcase that changes both the start attribute and the
> list-style-type css property.  Neither change has an effect on the list
> visually.
> 

With r19893 I'm seeing the type change. I think that was bug 12899 and it's fixed now.
Comment 7 Beth Dakin 2007-02-27 16:02:14 PST
I can confirm that this has been fixed. Marking as a duplicate.

*** This bug has been marked as a duplicate of 12899 ***
Comment 8 Justin Garcia 2007-02-27 16:33:03 PST
The problem still exists for changes to the start attribute.  Reopening.
Comment 9 Beth Dakin 2007-02-27 16:52:47 PST
I was too rash! You are so right, Justin, sorry I missed that. Note that this is *not* a regression from Safari 2.0.
Comment 10 Antti Koivisto 2007-02-27 18:24:07 PST
Created attachment 13416 [details]
update list markers when start attribute changes
Comment 11 Maciej Stachowiak 2007-02-27 18:48:17 PST
P1 because this is an important issue for Google.
Comment 12 Maciej Stachowiak 2007-02-27 22:04:33 PST
Comment on attachment 13416 [details]
update list markers when start attribute changes

r=me
Comment 13 Antti Koivisto 2007-02-28 08:00:10 PST
r19911