Fix notifications in queries, limit message highlights to channels

This commit is contained in:
Pavel Djundik 2016-05-16 22:46:22 +03:00
parent e6990e0fc7
commit 1c57363106
5 changed files with 14 additions and 12 deletions

View File

@ -868,9 +868,9 @@ button,
#chat .error, #chat .error,
#chat .error .from, #chat .error .from,
#chat .highlight .from, #chat .channel .highlight .from,
#chat .highlight .text, #chat .channel .highlight .text,
#chat .highlight .user { #chat .channel .highlight .user {
color: #f00; color: #f00;
} }

View File

@ -827,8 +827,7 @@ $(function() {
} }
} }
button = button.filter(":not(.active)"); if (button.hasClass("active")) {
if (button.length === 0) {
return; return;
} }

View File

@ -163,8 +163,8 @@ QUIT #d0907d
#chat .error, #chat .error,
#chat .error .from, #chat .error .from,
#chat .highlight, #chat .channel .highlight,
#chat .highlight .from { #chat .channel .highlight .from {
color: #f92772; color: #f92772;
} }

View File

@ -193,8 +193,8 @@ body {
#chat .error, #chat .error,
#chat .error .from, #chat .error .from,
#chat .highlight, #chat .channel .highlight,
#chat .highlight .from { #chat .channel .highlight .from {
color: #bc6c4c; color: #bc6c4c;
} }

View File

@ -46,7 +46,6 @@ module.exports = function(irc, network) {
if (data.type === Msg.Type.NOTICE) { if (data.type === Msg.Type.NOTICE) {
chan = network.channels[0]; chan = network.channels[0];
} else { } else {
highlight = !self;
chan = new Chan({ chan = new Chan({
type: Chan.Type.QUERY, type: Chan.Type.QUERY,
name: target name: target
@ -58,10 +57,14 @@ module.exports = function(irc, network) {
}); });
} }
} }
// Query messages (unless self) always highlight
if (chan.type === Chan.Type.QUERY) {
highlight = !self;
}
} }
// Query messages (unless self) always highlight // Self messages in channels are never highlighted
// Self messages are never highlighted
// Non-self messages are highlighted as soon as the nick is detected // Non-self messages are highlighted as soon as the nick is detected
if (!highlight && !self) { if (!highlight && !self) {
highlight = network.highlightRegex.test(data.message); highlight = network.highlightRegex.test(data.message);