[Webkit-unassigned] [Bug 55330] [Qt] [WK2] QGraphicsWKView::showContextMenu() should be public

Sun Feb 27 15:03:50 PST 2011

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

Kenneth Rohde Christiansen <kenneth at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #83993|review?                     |review+
               Flag|                            |

--- Comment #2 from Kenneth Rohde Christiansen <kenneth at webkit.org>  2011-02-27 15:03:50 PST ---
(From update of attachment 83993)
View in context: https://bugs.webkit.org/attachment.cgi?id=83993&action=review

Very nice to see documentation for our WebKit2 classes!

> Source/WebKit2/UIProcess/API/qt/qgraphicswkview.cpp:335
> +    actions possible for its current position, it is empty if there is no action for the position.

I don't think it should be a comma here. Maybe ; would do, or you could add another sentence instead.

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