Code review comment for lp:~osomon/webbrowser-app/uriboni-tab-context-menu

Riccardo Padovani (rpadovani) wrote :

I tested it extensively and looks good: I also looked to code and I think it's ok.

Only thing to fix: the 'x' to close a tab isn't clickable anymore, the MouseArea that intercepts the click for context menu overalls it. Except from that, for me it's ready to go

review: Needs Fixing

« Back to merge proposal