Separate input boxes for each view

NEW
Assigned to

Status

Other Applications
ChatZilla
--
enhancement
17 years ago
3 years ago

People

(Reporter: jwatt, Assigned: Robert Ginda)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win 9x 4.90; en-US; rv:0.9.5) Gecko/20011011
BuildID:    2001101117

As chatzilla stands one input box is used for all views that may be open for
different channels. Hence if text is typed into the input box and the user
switches to another view without hitting Enter, the text they entered is in the
input box for the new view. Since you never enter text for one view in the input
box of another it would seem to me to make more sence to have separate input
boxes for each view. This saves problems with text being entered in one view
then inadvertently sent to the wrong channel if the user changes view and sends
of some text to the new view without looking at the input box. It may sound
unlikely, but I almost did it, and it was a pain to have to delete the text I
had typed out but didn't want to post until the person it was directed to had
returned after temporarily leaving. 

Reproducible: Didn't try

Comment 1

16 years ago
id like a whole seperate window for each channel, rather than a tabbed view.  
which would also solve this problem.  
Perhaps i should post a whole new bug? with this one as a dependency?? i dunno

Comment 2

16 years ago
Alan, see bug 76175.

Comment 3

16 years ago
Remove myself from QA of 33 open Chatzilla bugs and change to default QA
contact, since I have no way to verify these easily.  Still no working Mozilla
on my primary platform and it doesn't look like it will happen anytime soon. :(
QA Contact: mozilla → samuel

Comment 4

15 years ago
*** Bug 195225 has been marked as a duplicate of this bug. ***

Comment 5

15 years ago
Created attachment 117678 [details] [diff] [review]
patch v1

Added an input history buffer for each view.

Updated

15 years ago
Attachment #117678 - Flags: review?(rginda)
(Assignee)

Comment 6

15 years ago
Comment on attachment 117678 [details] [diff] [review]
patch v1

>--- chatzilla.orig/content/chatzilla/lib/js/irc.js	2003-02-26 09:57:38.000000000 +0900
>+++ chatzilla/content/chatzilla/lib/js/irc.js	2003-03-18 22:16:35.000000000 +0900
>@@ -90,6 +90,8 @@
>     this.serverList = serverList;
>     this.eventPump = eventPump;
>     this.servers = new Object();
>+    this.inputHistory = new Array();
>+    this.lastHistoryReferenced = -1;
> 

This is crossing the line a bit.  irc.js is supposed to be the irc library, and
not know anything about the UI.

Personally, I think I prefer thte current behavior, so I'd like to see a pref
for this.  How about leaving client.inputHistory and
client.lastHistoryReferenced where they are, but save/restore them to the
currentObject when the tab changes.  Only if the pref is set.
(Assignee)

Updated

15 years ago
Attachment #117678 - Flags: review?(rginda) → review-

Comment 7

14 years ago
*** Bug 231857 has been marked as a duplicate of this bug. ***

Comment 8

14 years ago
(In reply to comment #6)
> Personally, I think I prefer thte current behavior, so I'd like to see a pref
> for this.

This might be hard to do with the floating & docking windows... I /could/ attach
a fixed input to each window, then the tabbed/docked views themselves wouldn't
have one, but it seems a bit odd.
Product: Core → Other Applications

Updated

9 years ago
QA Contact: samuel → chatzilla

Updated

3 years ago
Duplicate of this bug: 820768
You need to log in before you can comment on or make changes to this bug.