Hours of Operation Edit Note Field Check Boxes Don't Work Per Line

Bug #2036296 reported by Jennifer Pringle
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

Evergreen 3.11.1

When editing hours of operation (Administration -> Server Administration -> Organizational Units) if you click the check box for Edit Note Field beside a particular day of the week the note field opens up for all 7 days. Only the day of the week selected should have the note field display.

This doesn't prevent the feature from being used but the display is confusing as it's not what you expect to happen when you click only one check box.

Changed in evergreen:
status: New → Confirmed
Revision history for this message
Steven Mayo (stmayo) wrote :
tags: added: pullrequest
Changed in evergreen:
milestone: none → 3.next
milestone: 3.next → none
importance: Undecided → Medium
Revision history for this message
Gina Monti (gmonti90) wrote :
tags: added: signedoff
Revision history for this message
Galen Charlton (gmc) wrote :

The form is not that large; is there any real reason not to just always display the notes field, either as an additional column or underneath the main time inputs for each row? That way, all of the information about the hours is displayed without the user having to fiddle with checkboxes.

Revision history for this message
Terran McCanna (tmccanna) wrote :

I like that idea, too. My only concern with it being always visible would be the inclination of some people to think they need to fill in every field they see. If we go that route, maybe add an "(optional)" or something along with the label to make sure it's clear?

Revision history for this message
Galen Charlton (gmc) wrote :

I've created a working branch with an alternative patch to always display the note field input: user/gmcharlt/lp2036296_always_show_dow_note / https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/gmcharlt/lp2036296_always_show_dow_note

tags: removed: signedoff
Revision history for this message
Gina Monti (gmonti90) wrote :
tags: added: signedoff
Jason Boyer (jboyer)
Changed in evergreen:
milestone: none → 3.12.1
status: Confirmed → Fix Committed
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.