From: Manuel Nickschas <sputnick@quassel-irc.org>
Date: Tue, 30 Oct 2018 19:07:44 +0000 (+0100)
Subject: qtui: Disable web preview by default
X-Git-Tag: 0.13.0~8
X-Git-Url: https://git.quassel-irc.org/?p=quassel.git;a=commitdiff_plain;h=4d1f020e4e993bb72f08e6cf32d5e5a0eba51c9e

qtui: Disable web preview by default

Upon popular request, disable the web preview by default. This
feature tends to surprise unwitting new users that just happen
to hover over a questionable link, and the preview's functionality
is limited anyway with today's complex websites.

The web preview can still be enabled explicitly in chatview settings.
Existing configurations are unaffected by this change.
---

diff --git a/src/qtui/chatviewsettings.h b/src/qtui/chatviewsettings.h
index 1af17560..c491bbac 100644
--- a/src/qtui/chatviewsettings.h
+++ b/src/qtui/chatviewsettings.h
@@ -43,7 +43,7 @@ public:
     ChatViewSettings(ChatScene *scene);
     ChatViewSettings(ChatView *view);
 
-    inline bool showWebPreview() { return localValue("ShowWebPreview", true).toBool(); }
+    inline bool showWebPreview() { return localValue("ShowWebPreview", false).toBool(); }
     inline void enableWebPreview(bool enabled) { setLocalValue("ShowWebPreview", enabled); }
 
     /**
diff --git a/src/qtui/settingspages/chatviewsettingspage.ui b/src/qtui/settingspages/chatviewsettingspage.ui
index a4ab67ec..99f568a1 100644
--- a/src/qtui/settingspages/chatviewsettingspage.ui
+++ b/src/qtui/settingspages/chatviewsettingspage.ui
@@ -7,7 +7,7 @@
     <x>0</x>
     <y>0</y>
     <width>514</width>
-    <height>294</height>
+    <height>306</height>
    </rect>
   </property>
   <property name="windowTitle">
@@ -178,10 +178,10 @@
       <string>Show previews of webpages on URL hover</string>
      </property>
      <property name="checked">
-      <bool>true</bool>
+      <bool>false</bool>
      </property>
      <property name="defaultValue" stdset="0">
-      <bool>true</bool>
+      <bool>false</bool>
      </property>
      <property name="settingsKey" stdset="0">
       <string notr="true">ShowWebPreview</string>