Commit 29a9caf1 authored by Allan Sandfeld Jensen's avatar Allan Sandfeld Jensen
Browse files

Set settings declaratively instead of on events


Match the changes in quicktestbrowser in how settings are set.

Change-Id: I5b41de4c400c9c6802da20da4f9cc6f24f7db4da
Reviewed-by: default avatarAndras Becsi <andras.becsi@theqtcompany.com>
Showing with 4 additions and 3 deletions
......@@ -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()
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment