webbrowser-app

Code review comment for lp://staging/~abreu-alexandre/webbrowser-app/context-menu-to-overlay-webviews

  1. context-menu-to-overlay-webviews
  2. Merge into trunk
Revision history for this message
Olivier Tilloy (osomon) wrote :

The copyright year in the header for src/app/webcontainer/WebappWebview.qml should be 2016.

It would be good to add autopilot tests to verify that the menu also works in overlays.

The changes otherwise look good (and I’ve verified they don’t introduce regressions in the browser).

« Back to merge proposal