Remove displayed brackets around sender nicks

Remove brackets around nick names for the display role.

Add brackets around sender when copying ChatLines.  Only apply when
Message::Type == Plain, though.

Remove brackets from default timestamp setting

diff --git a/src/qtui/chatmonitorfilter.cpp b/src/qtui/chatmonitorfilter.cpp
index e3dbd63..8050653 100644 (file)
--- a/src/qtui/chatmonitorfilter.cpp
+++ b/src/qtui/chatmonitorfilter.cpp
@@ -123,7 +123,7 @@ QVariant ChatMonitorFilter::data(const QModelIndex &index, int role) const
         QString sender = MessageFilter::data(index, ChatLineModel::EditRole).toString();
         fields << sender;
     }
         QString sender = MessageFilter::data(index, ChatLineModel::EditRole).toString();
         fields << sender;
     }

-    return QString("<%1>").arg(fields.join(":"));
+    return QString("%1").arg(fields.join(" "));

 }
 
 
 }
 
 

diff --git a/src/qtui/chatscene.cpp b/src/qtui/chatscene.cpp
index 2f14742..8a5f7d4 100644 (file)
--- a/src/qtui/chatscene.cpp
+++ b/src/qtui/chatscene.cpp
@@ -1026,9 +1026,18 @@ QString ChatScene::selection() const
         for (int l = start; l <= end; l++) {
             if (_selectionMinCol == ChatLineModel::TimestampColumn)
                 result += _lines[l]->item(ChatLineModel::TimestampColumn)->data(MessageModel::DisplayRole).toString() + " ";
         for (int l = start; l <= end; l++) {
             if (_selectionMinCol == ChatLineModel::TimestampColumn)
                 result += _lines[l]->item(ChatLineModel::TimestampColumn)->data(MessageModel::DisplayRole).toString() + " ";

-            if (_selectionMinCol <= ChatLineModel::SenderColumn)
-                result += _lines[l]->item(ChatLineModel::SenderColumn)->data(MessageModel::DisplayRole).toString() + " ";
-            result += _lines[l]->item(ChatLineModel::ContentsColumn)->data(MessageModel::DisplayRole).toString() + "\n";
+            if (_selectionMinCol <= ChatLineModel::SenderColumn) {
+                ChatItem *item = _lines[l]->item(ChatLineModel::SenderColumn);
+                if (item->chatLine()->msgType() == Message::Plain) {
+                    // Copying to plain-text, re-add the sender brackets
+                    result += QString("<%1> ").arg(item->data(MessageModel::DisplayRole)
+                                                   .toString());
+                } else {
+                    result += item->data(MessageModel::DisplayRole).toString() + " ";
+                }
+            }
+            result += _lines[l]->item(ChatLineModel::ContentsColumn)
+                    ->data(MessageModel::DisplayRole).toString() + "\n";

         }
         return result;
     }
         }
         return result;
     }

diff --git a/src/qtui/chatviewsettings.h b/src/qtui/chatviewsettings.h
index 637b23f..fe4e383 100644 (file)
--- a/src/qtui/chatviewsettings.h
+++ b/src/qtui/chatviewsettings.h
@@ -45,7 +45,7 @@ public:
     inline bool showWebPreview() { return localValue("ShowWebPreview", true).toBool(); }
     inline void enableWebPreview(bool enabled) { setLocalValue("ShowWebPreview", enabled); }
 
     inline bool showWebPreview() { return localValue("ShowWebPreview", true).toBool(); }
     inline void enableWebPreview(bool enabled) { setLocalValue("ShowWebPreview", enabled); }
 

-    inline QString timestampFormatString() { return localValue("TimestampFormat", "[hh:mm:ss]").toString(); }
+    inline QString timestampFormatString() { return localValue("TimestampFormat", " hh:mm:ss").toString(); }

     inline void setTimestampFormatString(const QString &format) { setLocalValue("TimestampFormat", format); }
 
     inline QString webSearchUrlFormatString() { return localValue("WebSearchUrlFormat", "https://www.google.com/search?q=%s").toString(); }
     inline void setTimestampFormatString(const QString &format) { setLocalValue("TimestampFormat", format); }
 
     inline QString webSearchUrlFormatString() { return localValue("WebSearchUrlFormat", "https://www.google.com/search?q=%s").toString(); }

diff --git a/src/uisupport/uistyle.cpp b/src/uisupport/uistyle.cpp
index 169f5cd..bee1f4b 100644 (file)
--- a/src/uisupport/uistyle.cpp
+++ b/src/uisupport/uistyle.cpp
@@ -813,7 +813,7 @@ QString UiStyle::StyledMessage::decoratedSender() const
 {
     switch (type()) {
     case Message::Plain:
 {
     switch (type()) {
     case Message::Plain:

-        return QString("<%1>").arg(plainSender()); break;
+        return QString("%1").arg(plainSender()); break;

     case Message::Notice:
         return QString("[%1]").arg(plainSender()); break;
     case Message::Action:
     case Message::Notice:
         return QString("[%1]").arg(plainSender()); break;
     case Message::Action: