https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11373

Charles Farmer <charles.far...@inlibro.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Failed QA                   |Needs Signoff

--- Comment #33 from Charles Farmer <charles.far...@inlibro.com> ---
Hi Katrin,

Following your comments, I made cosmetic adjustments to the previous patch:
    - <span> no longer present in alert dialog
        - I also toned down from 'exclamation mark' to a simple period
    - removed the dangling space before ':'
    - server side formatting was changed from sprintf to $Price
        - I also removed the formatting in the <input> fields. It made no sense
to me that formatting was used for data traveling between the page and the
server, unbeknown to the user.

About the behavior of the form: I agree with you, the UX should definitely be
more user-friendly. Now, I'm open to changes, but I'd rather implement
everything once we reach a consensus on how it should behave than doing
"flavors" of the same patch just for QA. Nothing personal ;)

I'll put this back to 'Needs signoff', for now, just to make it clear that I
want people testing this patch, but feel free to dial it back to "Failed QA" or
"In discussion". I'll be keeping a close eye on this.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to