Ticket #5506 (new defect)
Opened 5 years ago
Last modified 3 years ago
Confusing formatting for attachments in Timeline
| Reported by: | anonymous | Owned by: | cboos |
|---|---|---|---|
| Priority: | normal | Milestone: | next-major-0.1X |
| Component: | timeline | Version: | devel |
| Severity: | minor | Keywords: | attachment |
| Cc: | |||
| Release Notes: | |||
| API Changes: | |||
Description
Simple solution can be done with JavaScript (attached).
Attachments
Change History
Changed 5 years ago by anonymous
- Attachment timeline_links_in_attachments.diff added
comment:1 Changed 5 years ago by cboos
- Keywords attachment needinfo added
- Milestone 0.11 deleted
- Severity changed from normal to minor
comment:2 follow-up: ↓ 3 Changed 5 years ago by anonymous
Alternative is to change formatting and keep 'Ticket #1234' as plain text (now it looks like a link)
comment:3 in reply to: ↑ 2 Changed 5 years ago by eblot
Replying to anonymous:
Alternative is to change formatting and keep 'Ticket #1234' as plain text (now it looks like a link)
I'd go for this one: as the whole line is a link, the ticket number (or the name of the page for wiki attachments...) should not appear as a link. However, they (ticket number, wiki page name) should nevertheless be highlighted, which another style than wiki links.
I really don't think it is worth going down the Javascript or jQuery road here.
comment:4 Changed 5 years ago by cboos
- Keywords needinfo removed
- Milestone set to 0.11
comment:5 Changed 3 years ago by cboos
- Milestone changed from 0.11-retriage to 0.13



Well, I'm not sure this is pertinent, as the whole line gets highlighted when you hover the mouse on it, and it's clear that the entry is about an attachment. Going elsewhere (the ticket) when you click on the attachment entry might be seen confusing as well by other users.
Plus I don't think we should generate Javascript trickery from the controller. The HTML should be kept as clean as possible. A somewhat cleaner solution would be to add these callbacks using jQuery once the page is loaded. But the first point would still stand.
Other opinions?