qa: Remove lots of superfluous semicolons

[quassel.git] / src / qtui / chatviewsettings.h

diff --git a/src/qtui/chatviewsettings.h b/src/qtui/chatviewsettings.h
index 06fb547..19aed61 100644 (file)
--- a/src/qtui/chatviewsettings.h
+++ b/src/qtui/chatviewsettings.h
@@ -22,6 +22,7 @@
 #define CHATVIEWSETTINGS_H
 
 #include "qtuisettings.h"
+#include "uistyle.h"
 
 class ChatScene;
 class ChatView;
@@ -36,13 +37,13 @@ public:
         OptIn = 1,
         OptOut = 2
     };
-    Q_DECLARE_FLAGS(operationModes, OperationMode);
+    Q_DECLARE_FLAGS(operationModes, OperationMode)
 
     ChatViewSettings(const QString &id = "__default__");
     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); }
 
     /**
@@ -74,11 +75,21 @@ public:
     inline void setTimestampFormatString(const QString &format) { setLocalValue("TimestampFormat", format); }
 
     /**
-     * Gets if prefixmodes are shown before sender names
+     * Gets how prefix modes are shown before sender names
      *
-     * @returns True if sender prefixmodes enabled, otherwise false
+     * @returns SenderPrefixMode of what format to use for showing sender prefix modes
      */
-    inline bool showSenderPrefixes() { return localValue("ShowSenderPrefixes", false).toBool(); }
+    inline UiStyle::SenderPrefixMode SenderPrefixDisplay() {
+        return static_cast<UiStyle::SenderPrefixMode>(
+                    localValue("SenderPrefixMode",
+                               QVariant::fromValue<UiStyle::SenderPrefixMode>(
+                                   UiStyle::SenderPrefixMode::HighestMode)).toInt());
+        // Cast the QVariant to an integer, then cast that to the enum class.
+        // .canConvert<UiStyle::SenderPrefixMode>() returned true, but
+        // .value<UiStyle::SenderPrefixMode>(); always gave the default value 0.
+        //
+        // There's probably a cleaner way of doing this.  I couldn't find it within 4 hours, so...
+    }
 
     /**
      * Gets if brackets are shown around sender names