Merge 1bdb118391 into f66bc37368

src/widgets/helper/ChannelView.hpp

@@ -180,7 +180,7 @@ public:
     LimitedQueueSnapshot<MessageLayoutPtr> &getMessagesSnapshot();
 
     // Returns true if message should be included
-    bool shouldIncludeMessage(const MessagePtr &message) const;
+    bool shouldIncludeMessage(const MessagePtr &m) const;
 
     void queueLayout();
     void invalidateBuffers();
@@ -216,11 +216,6 @@ public:
     Scrollbar *scrollbar();
 
     using ChannelViewID = std::size_t;
-    ///
-    /// \brief Get the ID of this ChannelView
-    ///
-    /// The ID is made of the underlying channel's name
-    /// combined with the filter set IDs
     ChannelViewID getID() const;
 
     pajlada::Signals::Signal<QMouseEvent *> mouseDown;

src/widgets/helper/NotebookTab.cpp

@@ -597,10 +597,11 @@ bool NotebookTab::shouldMessageHighlight(const ChannelView &channelViewSource,
         const auto &visibleSplits = visibleSplitContainer->getSplits();
         for (const auto &visibleSplit : visibleSplits)
         {
-            if (channelViewSource.getID() ==
-                    visibleSplit->getChannelView().getID() &&
+            if (channelViewSource.underlyingChannel() ==
+                    visibleSplit->getChannel() &&
                 visibleSplit->getChannelView().shouldIncludeMessage(message) &&
-                channelViewSource.shouldIncludeMessage(message))
+                channelViewSource.shouldIncludeMessage(message) &&
+                channelViewSource.getFilterIds().empty())
             {
                 return false;
             }

src/widgets/helper/NotebookTab.hpp

@@ -126,11 +126,6 @@ private:
                                 const MessagePtr &message) const;
 
     struct HighlightSources {
-        // Source of the update to the highlight status is the split
-        // in which a message that causes the update appears.
-        // This message can be just a plain message causing the
-        // HighlightState::NewMessage state, or a message that pings the user
-        // causing the HighlightState::Highlighted state
         std::unordered_set<ChannelView::ChannelViewID> newMessageSource;
         std::unordered_set<ChannelView::ChannelViewID> highlightedSource;