Support for editing notes in Serial Control

Bug #796795 reported by Dan Wells
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Dan Wells

Bug Description

The inability to edit notes in Serial Control has proven to be a pretty major usability hindrance. The current 'add' interface should be extended to allow for edits.

Tags: pullrequest
Revision history for this message
Dan Wells (dbw2) wrote :

This feature is now available in user/dbwells/lp-796795_serial_note_improvements for question, comment, or committal.

Dan

tags: added: pullrequest
Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

Hey Dan, this seemed to work great when I tried it. I made what I think is a tiny aesthetic improvement if you want to try it: it shows one "close window" button for the window rather than one per note.

collab/senator/lp-796795_serial_note_improvements

Revision history for this message
Dan Wells (dbw2) wrote :

Hello Lebbeous,

Thank you for the review and also for your contribution. Your code is a definite improvement.

I have pushed a further tweak based of your commit to the contrib branch you set up. This change restores the overall space and position of the close button while keeping the other improvements (appears once, shows even with no notes).

If you think it works, please feel free to push this all along. Otherwise, more changes are always welcome.

Thanks again,
Dan

Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

Thanks Dan. It works nicely. Pushed to master as 0c2d1a7a9b.

I'm being conservative about backporting since 2.1 is in beta. I doubt it would really hurt to backport, but leaving that decision to others.

Changed in evergreen:
status: In Progress → Fix Committed
Changed in evergreen:
milestone: none → 2.2.0
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.