Strike for ticket comment links

[Thijs Triemstra]

A link to a comment in a ticket (comment:16:ticket:5120) does not strike the link when the ticket is closed.

For example:

with comment vs ticket closed without comment.

[Remy Blank]

That's pretty easy to fix.

[trac@…]

ticket/api.py

@@ -493 +493 @@
             href = "%s#comment:%s" % (formatter.href.ticket(resource.id), cnum)
             title = _("Comment %(cnum)s for Ticket #%(id)s", cnum=cnum,
                       id=resource.id)
+            if 'TICKET_VIEW' in formatter.perm(resource):
+                for status in self.env.db_query(
+                    "SELECT status FROM ticket WHERE id=%s",
+                    (str(resource.id),)):
+                    return tag.a(label, href=href, title=title, _class=status)
             return tag.a(label, href=href, title=title)
         else:
             return label

This patch only strikes the link if you actually have the permission to see the ticket state — otherwise you could gain information you should not be allowed to know.

[Eli Carter]

fix 2 bugs, add regression test

[Eli Carter]

Committed to trunk in [10643]. Thanks to Radomir Dopieralski for the initial patch.

[Ryan J Ollos <ryano@…>]

While is seems correct to have comment:1:ticket:9981 display with strike-through, I'm not sure it makes sense for comment:1 to display with strike-through.

One effect of this that I noticed while commenting on #8295 is that the usernames shown at the top of quoted text when generating replyto comments are displayed in strike-through text, when the ticket is closed.

[Remy Blank]

Replying to Ryan J Ollos <ryano@…>:

While is seems correct to have comment:1:ticket:9981 display with strike-through, I'm not sure it makes sense for comment:1 to display with strike-through.

That makes sense, and is pretty easy to fix. Would you mind posting a patch?

[Christian Boos]

I think there's a problem with the strike-through style used for closed on text links in general, not only here for ticket comments, but also for closed milestones. Using strike-through is pretty much a convention for ticket numbers, but even those are sometimes hard to read when striked through: #8808…

I remember having experimented different alternatives at that time I introduced closed support for milestone (r5436), and I'm still somewhat unsatisfied with the current status. One alternative would be to use a border: #8808, milestone:0.12 (the border "encloses" the link…).

We could also add a background: #8808, milestone:0.12.

Or have only the background: #8808, milestone:0.12. This is a kind of strike-through but with a highlighter pen ;-)

[Remy Blank]

I agree that strike-through is difficult to read, and should be removed (pretty much everywhere in Trac). The drawback of the background-only solution is that it won't be visible when printing. The same would apply if we showed an icon next to the link. The border would highlight the link compared to others, when in principle we would rather un-highlight it (assuming closed tickets are less important that open ones).

Difficult problem indeed ;)

[Christian Boos]

What about simply: #8808, milestone:0.12?

[Dirk Stöcker]

Please not. Even if strike-through may be hard to read sometimes, the presented alternatives are much harder to understand. So when you change that, please at least allow each Trac maintainer to simply use the old method. Usually the fact, that a ticket is closed is much more important, than its number.

[Ryan J Ollos <ryano@…>]

Replying to rblank:

That makes sense, and is pretty easy to fix. Would you mind posting a patch?

Yes, I can take a crack at it by this weekend. The discussion about changing the text style for closed tickets is interesting, but am I correct to think that either way, references to comments within tickets should not provide any ticket status indication, and therefore the patch is needed and will be the same regardless of any changes to text styling for closed tickets?

I sort of like the red highlight background for closed ticket, but I'm not sure it is obvious enough. Then again, I might just be so used to the strike-through that nothing else will seem quite as intuitive.

[Remy Blank]

Replying to Ryan J Ollos <ryano@…>:

am I correct to think that either way, references to comments within tickets should not provide any ticket status indication, and therefore the patch is needed and will be the same regardless of any changes to text styling for closed tickets?

I think so, yes. Unless others have strong opinions against it.

[Christian Boos]

See follow-up in #10741.

[Ryan J Ollos <ryano@…>]

Replying to rblank:

I think so, yes. Unless others have strong opinions against it.

I've encountered a number of issues while trying to implement this ticket, so I've created a new dedicated ticket for the work: #10742. I think the change is still relevant even in the context of the changes proposed in #10012.