Edgewall Software
Modify

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

Click on 'Ticket #1234' at the attachment line on Timeline page opens attachment page and not actual ticket as expected.

Simple solution can be done with JavaScript (attached).

Attachments

timeline_links_in_attachments.diff (944 bytes) - added by anonymous 5 years ago.

Download all attachments as: .zip

Change History

Changed 5 years ago by anonymous

comment:1 Changed 5 years ago by cboos

  • Keywords attachment needinfo added
  • Milestone 0.11 deleted
  • Severity changed from normal to minor

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?

comment:2 follow-up: 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

Maybe this could be reworked together with #3132, so that we could allow for multiple links in a timeline entry's title (same as CvsTrac's timeline).

comment:5 Changed 3 years ago by cboos

  • Milestone changed from 0.11-retriage to 0.13
View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as new
as The resolution will be set. Next status will be 'closed'
to The owner will be changed from cboos. Next status will be 'new'
The owner will be changed from cboos to anonymous. Next status will be 'assigned'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.