364065 - Disabling CC list <select> for logged-out users can make the list unscrollable

Status: RESOLVED FIXED

(Bugzilla :: User Interface, defect, P1)

Description

[Jesse Ruderman]

Some buggy browsers (e.g. IE7, WebKit trunk) don't let you scroll a <select disabled size="5">.  To work around this bugginess, Bugzilla should not use the disabled attribute for CC lists (for logged-out users).

Example: https://bugzilla-test.mozilla.org/show_bug.cgi?id=12345

Comment 1

[Jesse Ruderman]

I filed a bug against WebKit: http://bugs.webkit.org/show_bug.cgi?id=11852.

Comment 2

[Frédéric Buclin]

(In reply to comment #0)
> disabled size="5">.  To work around this bugginess, Bugzilla should not use the
> disabled attribute for CC lists (for logged-out users).

I disagree. Logged out users don't need to be able to scroll the CC list. Disabling the CC list is consistent with how other fields work.

Comment 3

[Jesse Ruderman]

> Logged out users don't need to be able to scroll the CC list.

Logged out users should be able to see who is CCed.

> Disabling the CC list is consistent with how other fields work.

Disabling other fields doesn't prevent users from seeing their contents.

Comment 4

[Max Kanat-Alexander]

I actually *would* release Bugzilla 3.0 with this bug, so I'm not marking it a blocker. However, I would very much like to see it fixed before we release 3.0, if at all possible.

Comment 5

[Reed Loden [:reed]]

Attachment: patch - v1

Just remove the "disabled" option.

Comment 6

[Olav Vitters]

Comment on attachment 248969 [details] [diff] [review]
patch - v1

Makes sense. Do not think someone would be confused by this.

Comment 7

[Reed Loden [:reed]]

Checking in template/en/default/bug/edit.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/edit.html.tmpl,v  <--  edit.html.tmpl
new revision: 1.94; previous revision: 1.93
done