Ticket #2703: reply_to_description_static_buttons.diff

htdocs/css/ticket.css

@@ -39 +39 @@
 #ticket table.properties td { width: 30% }
 #ticket table.properties .description { border-top: 1px solid #dd9 }
 
+#ticket .description form { 
+ float: right;
+ position: relative;
+ bottom: 1.8em;
+}
+
 #changelog { border: 1px outset #996; padding: 1em }
 #changelog h3 {
  border-bottom: 1px solid #d7d7d7;
@@ -47 +53 @@
  font-weight: normal;
 }
 #changelog .threading {
-        float: right;
+ float: right;
+ position: relative;
+ bottom: 0.3em;
 }
+#changelog h3 .threading form { 
+ display: inline
+}
 .inlinebuttons input { 
- float: right;
  font-size: 70%;
  border-width: 1px;
- margin: 0 .5em .1em 1.5em;
+ margin: 0 0 0 .5em;
  padding: 0.1em;
- background-color: #f6f6f6;
+ background: none;
 }
-.inlinebuttons > input { /* rule ignored by IE */
- visibility: hidden;
-}
-div.change:hover .inlinebuttons input {
- visibility: visible;
-}
 
 #changelog .changes { list-style: square; margin-left: 2em; padding: 0 }
 #changelog .comment { margin-left: 2em }

templates/ticket.cs

@@ -73 +73 @@
  </table>
  <?cs if:ticket.description ?><div class="description">
   <?cs var:ticket.description.formatted ?>
+  <form method="get" action="<?cs var:ticket.href ?>#comment"><div class="inlinebuttons">
+   <input type="hidden" name="replyto" value="description" />
+   <input type="submit" value="Reply" title="Reply, quoting this description" /></div>
+  </form>
  </div><?cs /if ?>
 </div>
 
@@ -93 +97 @@
     set:nreplies = len(ticket.replies[change.cnum]) ?><?cs
     if:nreplies || change.replyto ?><span class="threading"> (<?cs
      if:change.replyto ?>in reply to: <?cs 
-      call:commentref('&uarr;', change.replyto) ?><?cs if nreplies ?>; <?cs /if ?><?cs
+      call:commentref('&uarr;&nbsp;', change.replyto) ?><?cs if nreplies ?>; <?cs /if ?><?cs
      /if ?><?cs
      if nreplies ?><?cs
       call:plural('follow-up', nreplies) ?>: <?cs 
       each:reply = ticket.replies[change.cnum] ?><?cs 
-       call:commentref('&darr;', reply) ?><?cs 
+       call:commentref('&darr;&nbsp;', reply) ?><?cs 
       /each ?><?cs 
      /if ?>)<?cs
-    /if ?></span><?cs
+    /if ?><form method="get" action="<?cs var:ticket.href ?>#comment"><span class="inlinebuttons">
+    <input type="hidden" name="replyto" value="<?cs var:change.cnum ?>" />
+    <input type="submit" value="Reply" title="Reply to comment <?cs var:change.cnum ?>" /></span>
+   </form>
+    </span><?cs
    /if ?><?cs
    var:change.date ?> changed by <?cs var:change.author ?><?cs
    if:change.cnum ?>&nbsp;<a href="#comment:<?cs var:change.cnum ?>" class="anchor"
       title="Permalink to comment:<?cs var:change.cnum ?>">&para;</a><?cs
    /if ?>
   </h3><?cs
-  if:change.cnum ?>
-   <form method="get" action="<?cs var:ticket.href ?>#comment"><div class="inlinebuttons">
-    <input type="hidden" name="replyto" value="<?cs var:change.cnum ?>" />
-    <input type="submit" value="Reply" title="Reply to comment <?cs var:change.cnum ?>" /></div>
-   </form><?cs 
-  /if ?><?cs
   if:len(change.fields) ?>
    <ul class="changes"><?cs
    each:field = change.fields ?>

trac/ticket/web_ui.py

@@ -574 +574 @@
 
         # -- Ticket Change History
 
+        def quote_original(author, original):
+            if not 'comment' in req.args: # i.e. the comment was not yet edited
+                req.hdf['ticket.comment'] = '\n'.join(
+                    ['Replying to [comment:%s %s]:' % (replyto, author)] +
+                    ['> %s' % line for line in original.splitlines()] + [''])
+
+        if replyto == 'description':
+            quote_original(reporter_id, ticket['description'])
+
         changelog = ticket.get_changelog(db=db)
         autonum = 0 # used for "root" numbers
         replies = {}
@@ -604 +613 @@
                         else:
                             this_num = old
                     assert this_num == str(autonum)
-                    # if we replied to this comment, quote it (with '>' prefix)
-                    if replyto == this_num and not 'comment' in req.args:
-                        req.hdf['ticket.comment'] = '\n'.join(
-                            ['Replying to [comment:%s %s]:' % \
-                             (replyto, author)] +
-                            ['> %s' % line for line in new.splitlines()] + [''])
+                    if replyto == this_num:
+                        quote_original(author, new)
             elif field == 'description':
                 current['fields'][field] = ''
             else: