Bug 19620
Summary: | REGRESSION: GoogleDocs: Font size change doesn't work first time | ||
---|---|---|---|
Product: | WebKit | Reporter: | Justin Garcia <justin.garcia> |
Component: | HTML Editing | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED INVALID | ||
Severity: | Normal | CC: | futurama, jparent |
Priority: | P2 | Keywords: | GoogleBug, Regression |
Version: | 528+ (Nightly build) | ||
Hardware: | Mac | ||
OS: | OS X 10.5 |
Justin Garcia
Go to http://docs.google.com/
Create a new document
Type "hello"
Select All
Change the font size from 10pt to 12pt
The font size doesn't change and the select becomes a caret at the beginning of the document. Select All again and change from 10pt to 12pt. The font size changes but you can no longer select text in the document.
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Justin Garcia
The second part (the text becoming unselectable was because of a user style sheet I had enabled to test for another bug. But the first part (the font size not changing and the selection becoming a caret) is a real bug.
Justin Garcia
Seems like a bug in GoogleDocs. On the first font size change they are:
Inserting an empty text node before the text node that contains "hello"
Clearing the selection with Selection::removeAllRanges()
Setting a selection with Selection::addRange(r) where r is [emptyTextNode, 0] to [emptyTextNode, 0]
Julie Parent
Yup, they are aware. Fix should go out with their next push.
Maciej Stachowiak
Closing as INVALID since this turned out to be a Google bug and was fixed on that end.