Add setting to prevent or highlight message overflow

CHANGELOG.md

@@ -38,6 +38,7 @@
 - Minor: Show picked outcome in prediction badges. (#3357)
 - Minor: Add support for Emoji in IRC (#3354)
 - Minor: Moved `/live` logs to its own subdirectory. (Logs from before this change will still be available in `Channels -> live`). (#3393)
+- Minor: Add setting to limit message input length (#3418)
 - Bugfix: Fix Split Input hotkeys not being available when input is hidden (#3362)
 - Bugfix: Fixed colored usernames sometimes not working. (#3170)
 - Bugfix: Restored ability to send duplicate `/me` messages. (#3166)

src/singletons/Settings.hpp

@@ -105,6 +105,8 @@ class Settings : public ABSettings, public ConcurrentSettings
     //    {"/appearance/messages/collapseLongMessages", false};
     IntSetting collpseMessagesMinLines = {
         "/appearance/messages/collapseMessagesMinLines", 0};
+    BoolSetting enforceMaxMessageLength = {
+        "/appearance/messages/enforceMaxMessageLength", false};
     BoolSetting alternateMessages = {
         "/appearance/messages/alternateMessageBackground", false};
     FloatSetting boldScale = {"/appearance/boldScale", 63};

src/widgets/helper/ResizingTextEdit.cpp

@@ -3,6 +3,7 @@
 #include "common/Common.hpp"
 #include "common/CompletionModel.hpp"
 #include "singletons/Settings.hpp"
+#include "widgets/splits/SplitInput.hpp"
 
 #include <QMimeData>
 #include <QMimeDatabase>
@@ -118,6 +119,13 @@ void ResizingTextEdit::keyPressEvent(QKeyEvent *event)
 
     this->keyPressed.invoke(event);
 
+    if (getSettings()->enforceMaxMessageLength.getValue() &&
+        this->toPlainText().size() >= SplitInput::TWITCH_MESSAGE_LIMIT &&
+        !keyEventIsNav(event))
+    {
+        return;
+    }
+
     bool doComplete =
         (event->key() == Qt::Key_Tab || event->key() == Qt::Key_Backtab) &&
         (event->modifiers() & Qt::ControlModifier) == Qt::NoModifier &&
@@ -191,6 +199,18 @@ void ResizingTextEdit::keyPressEvent(QKeyEvent *event)
     }
 }
 
+bool ResizingTextEdit::keyEventIsNav(QKeyEvent *event)
+{
+    const bool hasNavKey =
+        std::any_of(navKeys.cbegin(), navKeys.cend(), [event](int navKey) {
+            return event->key() == navKey;
+        });
+    const bool hasModifier = event->modifiers().testFlag(Qt::ControlModifier) ||
+                             event->modifiers().testFlag(Qt::MetaModifier);
+
+    return hasModifier || hasNavKey;
+}
+
 void ResizingTextEdit::focusInEvent(QFocusEvent *event)
 {
     QTextEdit::focusInEvent(event);
@@ -305,6 +325,22 @@ void ResizingTextEdit::insertFromMimeData(const QMimeData *source)
         }
     }
 
+    if (getSettings()->enforceMaxMessageLength.getValue())
+    {
+        const int remainingSize =
+            SplitInput::TWITCH_MESSAGE_LIMIT - this->toPlainText().size();
+
+        if (remainingSize == 0)
+        {
+            return;
+        }
+        else if (source->text().size() > remainingSize)
+        {
+            insertPlainText(source->text().left(remainingSize));
+            return;
+        }
+    }
+
     insertPlainText(source->text());
 }
 

src/widgets/helper/ResizingTextEdit.hpp

@@ -44,6 +44,12 @@ class ResizingTextEdit : public QTextEdit
     bool completionInProgress_ = false;
 
     bool eventFilter(QObject *widget, QEvent *event) override;
+
+    bool keyEventIsNav(QKeyEvent *event);
+    const std::vector<Qt::Key> navKeys{
+        Qt::Key_Backspace, Qt::Key_Delete,   Qt::Key_Home, Qt::Key_End,
+        Qt::Key_PageUp,    Qt::Key_PageDown, Qt::Key_Left, Qt::Key_Up,
+        Qt::Key_Right,     Qt::Key_Down};
 private slots:
     void insertCompletion(const QString &completion);
 };

src/widgets/settingspages/GeneralPage.cpp

@@ -258,6 +258,8 @@ void GeneralPage::initLayout(GeneralPageView &layout)
         [](auto args) {
             return fuzzyToInt(args.value, 0);
         });
+    layout.addCheckbox("Prevent typing past maximum message length",
+                       s.enforceMaxMessageLength);
     layout.addSeperator();
     layout.addCheckbox("Draw a line below the most recent message before "
                        "switching applications.",

src/widgets/splits/SplitInput.cpp

@@ -27,7 +27,6 @@
 #include <QPainter>
 
 namespace chatterino {
-const int TWITCH_MESSAGE_LIMIT = 500;
 
 SplitInput::SplitInput(Split *_chatWidget)
     : BaseWidget(_chatWidget)

src/widgets/splits/SplitInput.hpp

@@ -32,6 +32,8 @@ class SplitInput : public BaseWidget
     QString getInputText() const;
     void insertText(const QString &text);
 
+    static const int TWITCH_MESSAGE_LIMIT = 500;
+
     pajlada::Signals::Signal<const QString &> textChanged;
 
 protected: