Modify ↓
Opened 19 years ago
Closed 19 years ago
#4312 closed defect (fixed)
Attachment to ticket in timeline lacks title for ticket reference
| Reported by: | Owned by: | Christian Boos | |
|---|---|---|---|
| Priority: | normal | Milestone: | 0.11 |
| Component: | timeline | Version: | devel |
| Severity: | trivial | Keywords: | attachment |
| Cc: | Branch: | ||
| Release Notes: | |||
| API Changes: | |||
| Internal Changes: | |||
Description
For example, I see on the timeline right now <em>timeline_spelling.patch</em> attached to ticket <em>#4310</em>. I'm used to having a tooltip over things on the timeline that have a ticket reference, like for ticket changes, Ticket <em title="Ticket Title">#4310</em>....
This patch does it just for tickets by modifying the display callback. I'm not sure about the speed penalty.
-
trac/ticket/web_ui.py
545 545 # Attachments 546 546 if 'ticket_details' in filters: 547 547 def display(id): 548 return html('ticket ', html.em('#', id ))548 return html('ticket ', html.em('#', id, title=Ticket(self.env, id)['summary'])) 549 549 att = AttachmentModule(self.env) 550 550 for event in att.get_timeline_events(req, db, 'ticket', 551 551 start, stop, display):
Observed on t.e.o and reproduced on trunk.
Attachments (0)
Change History (2)
comment:1 by , 19 years ago
| Keywords: | attachment added |
|---|---|
| Milestone: | → 0.11 |
| Owner: | changed from to |
| Severity: | normal → trivial |
Note:
See TracTickets
for help on using tickets.



Well, this introduces no more overhead than if that would have been a regular comment for that ticket, so I guess it's OK.