You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bloodhound.apache.org by Apache Bloodhound <de...@bloodhound.apache.org> on 2014/03/03 08:08:18 UTC

Re: [Apache Bloodhound] #727: Handle long free text entries in sending qct fields to newticket interface

#727: Handle long free text entries in sending qct fields to newticket interface
--------------------------+-----------------------
  Reporter:  gjm          |      Owner:  antony
      Type:  enhancement  |     Status:  accepted
  Priority:  major        |  Milestone:  Release 8
 Component:  dashboard    |    Version:
Resolution:               |   Keywords:
--------------------------+-----------------------

Comment (by rjollos):

 Sorry that I didn't get to review this sooner. There is a regressions in
 r1570616: the product at scope is no longer the default selection.

 The best way I could see to fix this is with some refactoring, so I hope
 it doesn't seem like I'm trying to hijack your ticket, but since we are
 nearing a release it's probably easiest if I just push the fix and ask for
 feedback (particularly testing).

 What do you think about having the //More fields// link visible but
 disabled, rather than initially hidden? Actually, the only case that it
 will be hidden on page load is at global scope when `[ticket]
 default_product` hasn't been set, so it's probably not worth spending too
 much time refining.

 Minor things:
  * Python source code should wrap on 79 characters. Additional style
 guidelines that we follow can be found at trac:TracDev/CodingStyle.
  * !JavaScript code uses two-space indentation.

-- 
Ticket URL: <https://issues.apache.org/bloodhound/ticket/727#comment:13>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound issue tracker