show patch icon or notification on the comments page

Bug #172507 reported by LaserJock
28
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Abel Deuring

Bug Description

Similar to bug #4780 it would be good to show some sort of icon or something to indicate that an attachment is a patch in the comments section of the bug report. Right now there is no indication at all that I could find.

Revision history for this message
Emmet Hikory (persia) wrote :

This information is available by clicking the edit link for the attachment, although I'd agree it'd be nice to see it on the main page.

Revision history for this message
Brian Murray (brian-murray) wrote :

Additionally, it would be useful to have the attachment identified as a patch in the "Bug attachments" portlet on the left hand side of the bug report. An example of the patch not being identified can be found in bug 114815.

Kjell Braden (afflux)
Changed in malone:
status: New → Confirmed
Jonathan Lange (jml)
tags: added: patch-tracking
Deryck Hodge (deryck)
Changed in malone:
status: Confirmed → Triaged
importance: Undecided → High
assignee: nobody → Abel Deuring (adeuring)
Revision history for this message
Deryck Hodge (deryck) wrote :

Tom and I discussed this at the recent UDS, and now Abel and I discussed it a bit more. Plans are to do this bug -- i.e. add an icon to distinguish patches from other bug attachments -- and also to break out patches into a separate portlet on the bug page.

So there will be a new "Patches" portlet added that sits above the "Bug Attachments" portlet on a bug page at the same time as we fix this bug.

Deryck Hodge (deryck)
Changed in malone:
status: Triaged → In Progress
Deryck Hodge (deryck)
tags: added: story-patch-project
tags: added: story-patch-report
removed: story-patch-project
Deryck Hodge (deryck)
Changed in malone:
milestone: none → 3.1.12
Revision history for this message
Deryck Hodge (deryck) wrote :

Fixed in devel r10032.

Changed in malone:
status: In Progress → Fix Committed
Deryck Hodge (deryck)
Changed in malone:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.