Closed Bug 412887 Opened 17 years ago Closed 17 years ago

Disable user commands submenu for multiple user selection

Categories

(Other Applications :: ChatZilla, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Mitch, Assigned: bugzilla-mozilla-20000923)

Details

(Whiteboard: [cz-0.9.81])

Attachments

(1 file, 1 obsolete file)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.8.1.11) Gecko/20071127 Firefox/2.0.0.11
Build Identifier: ChatZilla 0.9.80

User commands only work on a single selected user, so they should be disabled when multiple users are selected.

Reproducible: Always

Steps to Reproduce:
1. Select multiple users.
2. Observe the user commands submenu.
Actual Results:  
The user commands submenu is enabled.

Expected Results:  
The user commands submenu should have been disabled.
Attached patch Patch v1.0 (obsolete) — Splinter Review
This patch disables the user commands submenu and hides the user label when multiple users are selected.
Attachment #297676 - Flags: superreview?(silver)
Attachment #297676 - Flags: review?(gijskruitbosch+bugs)
Attachment #297676 - Flags: superreview?(silver)
Version: unspecified → Trunk
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment on attachment 297676 [details] [diff] [review]
Patch v1.0

Why are you hiding the user label if you're still showing the op and user commands?
Attachment #297676 - Flags: review?(gijskruitbosch+bugs) → review+
Comment on attachment 297676 [details] [diff] [review]
Patch v1.0

Oops. Holding off on r+/r- until you've explained that change.
Attachment #297676 - Flags: review+ → review?(gijskruitbosch+bugs)
Assignee: rginda → silver
Attachment #297676 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #297941 - Flags: review?(gijskruitbosch+bugs)
Attachment #297676 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 297941 [details] [diff] [review]
Show user count and disable user commands if count>1

r=gijs
Attachment #297941 - Flags: review?(gijskruitbosch+bugs) → review+
Checked in --> FIXED.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Whiteboard: [cz-0.9.81]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: