[Webkit-unassigned] [Bug 20780] when choosing an auto-completed value in a text input field the javascript 'change' event is not fired.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Nov 5 10:11:27 PST 2009


https://bugs.webkit.org/show_bug.cgi?id=20780





--- Comment #24 from Darin Adler <darin at apple.com>  2009-11-05 10:11:26 PDT ---
Typically it's best to address new issues in a separate bug. The naming issue
is not the one this bug describes. And the automated commit queue tool works
best when each bug is resolved with a single patch.

(In reply to comment #23)
> Renamed RenderTextControl::m_edited to m_wasEditedSinceLastChangeEvent

Maybe it should be m_wasChangedSinceLastChangeEvent. I’m not sure setting the
text of a control counts as "being edited".

> RenderTextControl::m_userEdited to m_wasLastEditedByAUser

I don’t think "a user" is good here. We normally leave our articles from our
names. I also think that we should not call setting the text "being edited".
Perhaps m_wasEditedByUser is all we need for a good name here. Or maybe
m_lastChangeWasUserEdit. Maybe there's a better idea for the name. But in
general I think we should call changes done by the HTML editing commands
"editing" and other changes not be called editing.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list