Open
Bug 293230
Opened 19 years ago
Updated 15 years ago
Do not copy "important" (stalk) messages to network tab when present and active in channel they occur in
Categories
(Other Applications :: ChatZilla, enhancement)
Other Applications
ChatZilla
Tracking
(Not tracked)
NEW
People
(Reporter: svl-bmo, Assigned: rginda)
Details
Attachments
(1 file, 1 obsolete file)
1.11 KB,
patch
|
Details | Diff | Splinter Review |
When I'm in a channel, actively talking to people, I do not want lines there that include my name to be copied to the network tab. Right now I'm constantly focusing this tab to clear the red label so that I _do_ notice when other important stuff comes in. I suggest that when: 1) The tab where the stalk match happens is focused. 2) You have been active in this tab in the last x seconds/minutes (configurable? probably better try to choose a sane value - something like 1 minute) as evidenced by having switched to this tab or entered input during that time. 3) /away isn't set. ...the stalk match will not be copied to the network tab. ReadMe thinks this should be configurable with several options ("don't copy from active channel, always copy when away"). I think my proposal is simply the way to go and that everyone would want it (plus follow the Hixie school of thought on this that you need a _really_ good reason to add more prefs), but in the end I'd be happy with either way, as long as the requested behaviour is possible.
Comment 1•19 years ago
|
||
I would love to not have to switch to the network view just to clear the red too, but there are some points. - we can't get an idle time comparable with native clients. - if we do decide to do a "ChatZilla idle time" (which I don't like), we must tread very carefully. - we currently supress some sound events, see here: http://lxr.mozilla.org/mozilla/source/extensions/irc/xul/content/static.js#2363 While I also agree adding prefs is generally bad, I'm not sure we will be able to do this one well enough to get away with none.
Comment 2•19 years ago
|
||
i've done a patch to not copy the messages if they are sent tot he active tab and you are not away. Any opinions on this before i request a review?
Comment 3•19 years ago
|
||
ok, now it still copies if the window isnt focused - and the comment explains whats going on.
Attachment #182861 -
Attachment is obsolete: true
Assignee | ||
Comment 4•19 years ago
|
||
I don't bother with /away, and I don't want to lose messages just because they happened to be on the active tab, so I'd have to give it a thumbs down. It also makes that expression far too difficult to parse... or even explain. 7 terms in a single expression is too much. I might agree with an "only while away" modifier to "copy important messages", though. It might surface in the UI as a submenu... Copy important messages... O Never * Always O Only while away The "copyMessages" pref could become a string that represented one of those three states. The code would have to deal with the old boolean pref gracefully, of course.
Comment 5•19 years ago
|
||
(In reply to comment #4) >... > Copy important messages... > O Never > * Always > O Only while away good choices. default should be always, which I assume you are indicating with the "*"
Updated•15 years ago
|
QA Contact: samuel → chatzilla
You need to log in
before you can comment on or make changes to this bug.
Description
•