488943 - The "Private" checkbox is meaningless to new users

Status: RESOLVED FIXED

(Bugzilla :: User Interface, defect)

Description

[Max Kanat-Alexander]

Pyrzak's students' research points out:

Private could mean that only the submitter sees the description or that only people assigned to work on the bug can see it, or something else entirely.  There’s no help to explain the implications of making a description private.

Comment 1

[Frédéric Buclin]

Would a tooltip help? I'm not sure "Confidential" would be a better name.

Comment 2

[Max Kanat-Alexander]

Attachment: v1

Okay, this fixes it everywhere except bug/comments.html.tmpl. I didn't change the word "Private" there because we need to use minimal UI space. I considered just putting the name of the insidergroup there instead--do you think that would be better?

Comment 3

[Guy Pyrzak]

Comment on attachment 373423 [details] [diff] [review]
v1

it's the right start, but I'd like to let people know that on the comments that it will be marked as private if they check the box or somethign like that. i don't think it needs to be wordy, maybe just say (private) after the text you already have if that makes sense. That way uses might learn what private means.

Comment 4

[Max Kanat-Alexander]

Attachment: v2

Oh, good idea! How about this?

Comment 5

[Guy Pyrzak]

Comment on attachment 379032 [details] [diff] [review]
v2

looks good. Although my runtests.sh failed (but not caused by anything in this code). I'd prefer not to use <br>'s to do new lines (css is better) but since it is already used all over I'm guessing that's another bug.

Comment 6

[Frédéric Buclin]

Comment on attachment 379032 [details] [diff] [review]
v2

>Index: template/en/default/attachment/edit.html.tmpl

>+            <label for="isprivate">private (only visible to 
>+              <strong>[% Param('insidergroup') FILTER html %]</strong>)
>+            </label>

For consistency with other places, the sentence should be "... to the foo group". Please fix that on checkin. r=LpSolit

Comment 7

[Max Kanat-Alexander]

(In reply to comment #6)
> For consistency with other places, the sentence should be "... to the foo
> group". Please fix that on checkin. r=LpSolit

  That's too long to fit well in the UI there.

Comment 8

[Frédéric Buclin]

(In reply to comment #7)
>   That's too long to fit well in the UI there.

There is a plenty of place. The requestee field for attachment flags already takes some place. And in the worst case, the text will wrap.

Comment 9

[Max Kanat-Alexander]

Checking in template/en/default/attachment/create.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/attachment/create.html.tmpl,v  <--  create.html.tmpl
new revision: 1.40; previous revision: 1.39
done
Checking in template/en/default/attachment/edit.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/attachment/edit.html.tmpl,v  <--  edit.html.tmpl
new revision: 1.60; previous revision: 1.59
done
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.158; previous revision: 1.157
done
Checking in template/en/default/bug/create/create.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/create/create.html.tmpl,v  <--  create.html.tmpl
new revision: 1.94; previous revision: 1.93
done
Checking in template/en/default/list/edit-multiple.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/list/edit-multiple.html.tmpl,v  <--  edit-multiple.html.tmpl
new revision: 1.56; previous revision: 1.55
done