Summary: | Selection changed by indent | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Julie Parent <jparent> | ||||||||||||
Component: | HTML Editing | Assignee: | Ryosuke Niwa <rniwa> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | eric, justin.garcia, rniwa | ||||||||||||
Priority: | P2 | ||||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||
Hardware: | PC | ||||||||||||||
OS: | OS X 10.5 | ||||||||||||||
Bug Depends on: | 27038 | ||||||||||||||
Bug Blocks: | |||||||||||||||
Attachments: |
|
Description
Julie Parent
2009-06-29 14:40:48 PDT
Yes, I have become aware of this problem right after submitting the patch. This is due to changeset 45316. The fact that indentRegion stores the index of selection and restores it later is to with selection being changed. I believe restoring selection is only needed because of what moveParagraph does. Right now, I'm trying to avoid using moveParagraph at all since this function is causing a lot of problems. Created attachment 32347 [details] Refactors indentRegion further, and fixes the bug 26816 and 25317 Comment on attachment 32347 [details] Refactors indentRegion further, and fixes the bug 26816 and 25317 This patch is rather large to review. I certainly can't do so tonight, and I think you'll have trouble finding a reviewer for a patch of this size. :( Created attachment 32367 [details]
Refactors indentRegion and fixes the bugs 26816 and 25317 (a minor change on the prototype of indentListItem)
Created attachment 32436 [details] fixes the bug This patch may be committed only after the patch for https://bugs.webkit.org/show_bug.cgi?id=27038 is committed. Created attachment 32487 [details]
change due to the resubmission for 27038
Comment on attachment 32487 [details]
change due to the resubmission for 27038
Should have a bug:
+ // FIXME: we need to deal with the case where there is no li (malformed HTML)
+ if (!selectedListItem->hasTagName(liTag))
+ return false;
024 \ No newline at end of file
Needs an arg name for the Node*:
59 void indentIntoBlockquote(const VisiblePosition&, const VisiblePosition&, RefPtr<Element>&, Node*);
Looks fine otherwise. Thanks!
Created attachment 33073 [details] 5th submission The followup bug is filed as the bug 27441. Comment on attachment 33073 [details]
5th submission
Looks OK.
Landed in http://trac.webkit.org/changeset/46142 |