From 29a9caf106e5ea82202b66f6cd45c2db85fb734b Mon Sep 17 00:00:00 2001
From: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
Date: Mon, 16 Mar 2015 11:17:38 +0100
Subject: Set settings declaratively instead of on events

Match the changes in quicktestbrowser in how settings are set.

Change-Id: I5b41de4c400c9c6802da20da4f9cc6f24f7db4da
Reviewed-by: Andras Becsi <andras.becsi@theqtcompany.com>
---
 examples/webengine/quicknanobrowser/BrowserWindow.qml | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

(limited to 'examples/webengine')

diff --git a/examples/webengine/quicknanobrowser/BrowserWindow.qml b/examples/webengine/quicknanobrowser/BrowserWindow.qml
index f61bce268..da599bfb4 100644
--- a/examples/webengine/quicknanobrowser/BrowserWindow.qml
+++ b/examples/webengine/quicknanobrowser/BrowserWindow.qml
@@ -78,6 +78,7 @@ ApplicationWindow {
     property bool platformIsMac: styleItem.style == "mac"
 
     Settings {
+        id : appSettings
         property alias autoLoadImages: loadImages.checked;
         property alias javaScriptEnabled: javaScriptEnabled.checked;
         property alias errorPageEnabled: errorPageEnabled.checked;
@@ -230,21 +231,18 @@ ApplicationWindow {
                             text: "Autoload images"
                             checkable: true
                             checked: WebEngine.settings.autoLoadImages
-                            onCheckedChanged: WebEngine.settings.autoLoadImages = checked
                         }
                         MenuItem {
                             id: javaScriptEnabled
                             text: "JavaScript On"
                             checkable: true
                             checked: WebEngine.settings.javascriptEnabled
-                            onCheckedChanged: WebEngine.settings.javascriptEnabled = checked
                         }
                         MenuItem {
                             id: errorPageEnabled
                             text: "ErrorPage On"
                             checkable: true
                             checked: WebEngine.settings.errorPageEnabled
-                            onCheckedChanged: WebEngine.settings.errorPageEnabled = checked
                         }
                         MenuItem {
                             id: offTheRecordEnabled
@@ -324,6 +322,9 @@ ApplicationWindow {
                         }
                     }
                 ]
+                settings.autoLoadImages: appSettings.autoLoadImages
+                settings.javascriptEnabled: appSettings.javaScriptEnabled
+                settings.errorPageEnabled: appSettings.errorPageEnabled
 
                 onCertificateError: {
                     error.defer()
-- 
cgit v1.2.3