Search Preferences: labels and form fields are not associated with each other

Bug #2036313 reported by Jane Sandberg
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
3.11
Fix Released
Medium
Unassigned

Bug Description

Steps to recreate:

1. Install aXe or WAVE browser extension
2. In the Web client, go to Search > Search the Catalog
3. Press the "Search Preferences" button
4. Run your browser extension
5. Note that:
    * Preferred Library combobox is associated with a whole bunch of labels
    * Default Search Pane combobox is not associated with its label
    * Search Results Per Page is not associated with its label
    * Exclude Electronic Resources checkbox is not associated with its label

Revision history for this message
Stephanie Leary (stephanieleary) wrote :

Branch here: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/sleary/lp2036313-search-prefs-form-labels

Adds <label> tags to the last three form fields on the search preferences screen:

* Default Search Pane
* Search Results Per Page
* Exclude Electronic Resources

To test, visit /eg2/en-US/staff/catalog/prefs (adjusting your locale) and run WAVE or Axe Dev Tools to get a report on form fields without labels.

With the branch for bug 1999158 in place, all form fields should be labeled. Without that patch, the Default Search Library and Preferred Library will be missing labels, but the three listed above will be correctly labeled.

tags: added: pullrequest ux-forms
Changed in evergreen:
milestone: none → 3.12-beta
Andrea Neiman (aneiman)
Changed in evergreen:
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
Ruth Frasur Davis (redavis) wrote :

I have run the Axe DevTools in both a patched and non-patched environment for comparison. I confirm that the form element label issues described in the original LP ticket have been corrected. I consent to add my sign off using my username, rfrasur, and email address, <email address hidden>.

tags: added: signedoff
Changed in evergreen:
assignee: nobody → Terran McCanna (tmccanna)
Revision history for this message
Terran McCanna (tmccanna) wrote :

Looks good! Thanks Stephanie and Ruth. Fix committed to main and pushed back to 3.11. (Did not apply cleanly to 3.10.)

Changed in evergreen:
status: Confirmed → Fix Committed
assignee: Terran McCanna (tmccanna) → nobody
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.