fields.html help should happen when hovering over field labels

RESOLVED FIXED in Bugzilla 4.0

Status

()

Bugzilla
Creating/Changing Bugs
--
enhancement
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Max Kanat-Alexander, Assigned: Max Kanat-Alexander)

Tracking

(Blocks: 1 bug)

Bugzilla 4.0
Dependency tree / graph
Bug Flags:
approval +

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

8 years ago
I was reading the HCI research results again (and I was talking a bit with pyrzak about this also) and I noticed that it's not just the lack of fields.html help for each field that was confusing, but the very fact that clicking on the links would take people to the help was confusing.

  I think that perhaps in addition to the links, we could have a tooltip for the field labels, and change the cursor to the "arrow with question mark" cursor.

  I don't want to use the Help System (which instantly positions overlays containing help) because it can be annoying and block off the form, whereas tooltips are (a) easy and (b) a normal and expected browser mechanism that doesn't cause trouble.
(Assignee)

Comment 1

8 years ago
Created attachment 435758 [details] [diff] [review]
v1

  Okay, this patch is a bit large because it does three or four things. If you want me to split it into three separate patches, just let me know. Here are all the things it does:

  1) Refactors the label code for fields into bug/field-label.html.tmpl
  2) Modifies field-label.html.tmpl to display a tooltip for anything that has help in help_html, and adds all custom fields to help_html.
  3) Switches the create.html.tmpl template to using field.html.tmpl for the bug_status field.
  4) Modifies create.html.tmpl to use field-label for all the fields that had a help link before.

  I'll do show_bug in a separate patch, since we haven't even implemented help links for everything on show_bug yet.
Assignee: create-and-change → mkanat
Status: NEW → ASSIGNED
Attachment #435758 - Flags: review?(LpSolit)

Comment 2

8 years ago
Comment on attachment 435758 [details] [diff] [review]
v1

The changes in enter_bug.cgi have nothing to do with the bug. They are not even a code cleanup (they just make it harder to track changes in Bonsai). Why doing these changes?
(Assignee)

Comment 3

8 years ago
  The enter_bug changes do have to do with the bug. In order for fields.html to work, we have to pass objects to the template, not strings.
(Assignee)

Updated

8 years ago
Attachment #435758 - Flags: review?(LpSolit) → review?(timello)
Comment on attachment 435758 [details] [diff] [review]
v1

The patch is bit-rotted. It does not apply in trunk. enter_bug.cgi has been changed.
Attachment #435758 - Flags: review?(timello) → review-

Updated

8 years ago
Blocks: 494395
(Assignee)

Comment 5

8 years ago
Created attachment 445054 [details] [diff] [review]
v2

Ah, thanks for catching that! Here's a fix for the bitrot.
Attachment #435758 - Attachment is obsolete: true
Attachment #445054 - Flags: review?(timello)

Updated

8 years ago
Attachment #445054 - Flags: review?(timello) → review-
Comment on attachment 445054 [details] [diff] [review]
v2

It is breaking runtest :(.

t/008filter.t.

Updated

8 years ago
Blocks: 565650
(Assignee)

Comment 7

8 years ago
Created attachment 445107 [details] [diff] [review]
v3

Ah, thanks! Fixed.
Attachment #445054 - Attachment is obsolete: true
Attachment #445107 - Flags: review?(timello)
Comment on attachment 445107 [details] [diff] [review]
v3

>=== modified file 'template/en/default/bug/create/create.html.tmpl'
>+    [% INCLUDE bug/field.html.tmpl
>+      bug = default, field = bug_fields.bug_status,
>+      editable = (bug_status.size > 1), value = default.bug_status
>+      override_legal_values = bug_status  %]

Nit: one space more before the '%]'.

The rest ok! Nice code change! Much better now!
Attachment #445107 - Flags: review?(timello) → review+

Updated

8 years ago
Flags: approval?
(Assignee)

Updated

8 years ago
Flags: approval? → approval+
(Assignee)

Comment 9

8 years ago
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/                       
modified enter_bug.cgi
modified skins/standard/global.css
modified template/en/default/filterexceptions.pl
modified template/en/default/bug/field-help.none.tmpl
added template/en/default/bug/field-label.html.tmpl
modified template/en/default/bug/field.html.tmpl
modified template/en/default/bug/create/create.html.tmpl
modified template/en/default/pages/fields.html.tmpl
Committed revision 7162.
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED

Updated

8 years ago
Blocks: 565790
You need to log in before you can comment on or make changes to this bug.