Trac Ticket Author Gravatar disappears when comment preview is created

[adamsilverstein]

When entering a response to a ticket in trac (Chrome Dev/Mac), I noticed that the Gravatar of the original ticket author disappears the first time a preview of my response is generated. I haven't investigated extensively, but ​here is a screencast of the issue. Note @jeremyfelt's Gravatar disappears when my comment preview is generated.

[adamsilverstein]

I tracked the issue down to this line:
$("#ticket").replaceWith(items.filter('#ticket'));

Here is what i think is happening - when the auto_preview.js fires its ajax callback, it hits the main url to get the page html, the reply from this call is missing the gravatar, so when the replaceWith fires, the gravatar is removed.

I discussed this issue with @nacin at #wcsea and he knows what the problem is and how to fix it.

[nacin]

This isn't a simple fix, so I'm just going to wontfix this, just because it's not a big deal.

[adamsilverstein]

Replying to nacin:

This isn't a simple fix, so I'm just going to wontfix this, just because it's not a big deal.

Ok, i will block the issue from my mind so I no longer see it :)

[SergeyBiryukov]

Replying to nacin:

This isn't a simple fix, so I'm just going to wontfix this, just because it's not a big deal.

When I'm writing an initial reply to a ticket, I often go the reporter's WP.org profile. The only link to the profile in that case is under the Gravatar, and when it disappears, it's a big deal for me :)

[nacin]

OK. Maybe we could just restore it with a little CSS. I'd rather not override the ticket_box.html template, though we could do that pretty sanely, too. I'll look.

[adamsilverstein]

Replying to SergeyBiryukov:

Replying to nacin:

This isn't a simple fix, so I'm just going to wontfix this, just because it's not a big deal.

When I'm writing an initial reply to a ticket, I often go to the reporter's WP.org profile. The only link to the profile in that case is under the Gravatar, and when it disappears, it's a big deal for me :)

Sergey,

Good point, I'm glad you brought that up.

I have to admit, the disappearing Gravatar wasn't something I was really going to be able to ignore :)

@nacin - if you want a JS solution/kludge that works without modifying anything server side, I can spin that up.

[nacin]

In 1998:

Add a pristine ticket_preview.html from Trac 1.0.1. see #418.

[nacin]

In 1999:

Trac: Ensure site.html changes are applied to ticket previews.

site.html is not loaded in this case. Thus, split off the functionality into site-ticket.html,
then load site-ticket.html in an overloaded ticket_preview.html.

fixes #418.

[nacin]

In revisiting this, I came up with a fairly elegant (low maintenance) way to fix it.

Fun fact, we actually rearrange the order of the fields in the ticket box. It wasn't as noticeable, but they used to revert to the default Trac order when a preview is run. :-)

[adamsilverstein]

Thanks for fixing this @nacin!