[Webkit-unassigned] [Bug 36292] The numbers are not displayed properly when entering through VKB in number mode
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Mar 18 07:21:53 PDT 2010
https://bugs.webkit.org/show_bug.cgi?id=36292
Simon Hausmann <hausmann at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #51020|review?, commit-queue? |review-, commit-queue-
Flag| |
--- Comment #2 from Simon Hausmann <hausmann at webkit.org> 2010-03-18 07:21:53 PST ---
(From update of attachment 51020)
I think in principle this looks good, but I can see two things missing:
* A ChangeLog entry
* A comment indicating that this is a missing feature in QGraphicsItem, ideally
with a link to an entry in the Qt JIRA tracking this missing feature. Then we
can remove this code again once it's fixed in Qt.
> +#if !defined(QT_NO_IM) && (defined(Q_WS_X11) || defined(Q_WS_QWS) || defined(Q_OS_SYMBIAN))
> + QList<QGraphicsView *> views = q->scene()->views();
Coding style, no space before the '*'
> + for (int c = 0; c < views.size(); ++c) {
> + QInputContext *ic = views[c]->inputContext();
I think you may want to use views.at(c) instead of [], to avoid detaching the
QList that the scene returns.
> + if (ic)
> + ic->update();
Shouldn't this be a call to views.at(c)->updateMicroFocus() instead?
--
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