Closed Bug 231621 Opened 21 years ago Closed 21 years ago

Add a middle-click action for the Chatzilla client window (open in new tab, configurable)

Categories

(Other Applications :: ChatZilla, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
mozilla1.7alpha

People

(Reporter: durbacher, Assigned: durbacher)

Details

Attachments

(1 file)

1.66 KB, patch
bugzilla-mozilla-20000923
: review+
Details | Diff | Splinter Review
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.6) Gecko/20040113
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.6) Gecko/20040113

Bug 230051 is a mixture of "middle-click -> new tab" and "chatzilla should use
Browser prefs for click actions/tabs".
The latter seems being discussed by the Chatzilla people, the first one seems to
be welcome, so I split that other bug.
This one is for middle click, the other one strictly for honoring Browser prefs.

Reproducible: Always

Steps to Reproduce:
1. Middle-click on URL in Chatzilla
Actual Results:  
Nothing.

Expected Results:  
Open a new browser tab.

Patch on the way.
Assignee: rginda → durbacher
Status: NEW → ASSIGNED
Attached patch patchSplinter Review
This adds a middle-click action/pref to chatzilla, defaulting to opening in new
tab.
Comment on attachment 139505 [details] [diff] [review]
patch

Requesting r= from Silver (we spoke about it on IRC).

I'm not familiar with chatzilla review requirements and checkin procedures.
Assuming your review is sufficient: Can anybody with cvs write access check
this in? Then I guess I have a person who'll do it. Else, could you please
check this into the tree?  Or will this go into the hacksrus chatzilla builds
first??
Attachment #139505 - Flags: review?(silver)
When reassigning, it's customary to put the previous assignee in the CC list, if
they're not already there.
Comment on attachment 139505 [details] [diff] [review]
patch

r=silver

Works like a charm.
Attachment #139505 - Flags: review?(silver) → review+
Comment on attachment 139505 [details] [diff] [review]
patch

Checked in.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
v
Status: RESOLVED → VERIFIED
Target Milestone: --- → mozilla1.7alpha
Product: Core → Other Applications
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: