Project

General

Profile

Actions

Patch #2607

closed

Avatars display in forums

Added by Brad Beattie almost 16 years ago. Updated over 14 years ago.

Status:
Closed
Priority:
Normal
Category:
Forums
Target version:
-
Start date:
2009-01-28
Due date:
% Done:

90%

Estimated time:

Description

Not sure what format you want your patches in, so I'll just include this here.

app/views/messages/show.rhtml from release 0.8.0, between lines 32 and 33

<%= avatar(message.user, :size => "32") %>


Files

application.css (22.3 KB) application.css public/stylesheets/application.css (modified from 0.8.0) Brad Beattie, 2009-01-29 22:50
show.rhtml (3.2 KB) show.rhtml app/views/messages/show.rhtml (modified from 0.8.0) Brad Beattie, 2009-01-29 22:50
application.css (22.4 KB) application.css public/stylesheets/application.css Brad Beattie, 2009-01-29 22:54
avatars.patch (1.58 KB) avatars.patch Brad Beattie, 2009-02-05 07:55

Related issues

Related to Redmine - Feature #4468: Show gravatar images in forum viewClosedJean-Philippe Lang2009-12-22

Actions
Actions #1

Updated by Brad Beattie almost 16 years ago

Kay, just went and tested this idea out. Two changes.

message.user didn't work, but message.author does. Change app/views/messages/show.rhtml between lines 32 and 33:

<%= avatar(message.author, :size => "32") %>

The avatars need to float left. Include .message img.gravatar in public/stylesheets/application.css on line 645

#history img.gravatar,
.message img.gravatar {
Actions #2

Updated by Brad Beattie almost 16 years ago

Hrm. On one-line posts, this can cause the subsequent message h4 to indent incorrectly.

Actions #3

Updated by Brad Beattie almost 16 years ago

Brad Beattie wrote:

Hrm. On one-line posts, this can cause the subsequent message h4 to indent incorrectly.

Well, clear: left is certainly an option, but that would require the contextual div to be included in the message div. That makes sense as that's how journal divs are working now and consistency is awesome like that.

Actions #5

Updated by Brad Beattie almost 16 years ago

  • Assignee set to Jean-Philippe Lang
  • % Done changed from 0 to 90

Okay, done. I just need someone to review it and integrate it if appropriate.

Actions #6

Updated by Brad Beattie almost 16 years ago

Minor clear: left fix.

Actions #7

Updated by Jean-Philippe Lang almost 16 years ago

Brad, you should submit your changes using a single patch file.
It's much more easy to review and apply.
You can generate a patch file using svn diff command.

Actions #8

Updated by Brad Beattie almost 16 years ago

Right you are, good sir. When I made this ticket I was still getting into the swing of things.

Actions #9

Updated by Oshyan Greene over 14 years ago

Is this planned for inclusion in a future version? The forums are a bit hard to read at the moment without visual poster identity context. This would help a lot. Thanks!

Actions #10

Updated by Felix Schäfer over 14 years ago

  • Status changed from New to Closed

As far as I can tell, this is a dupe of #4468, which has been fixed in 0.9.0.

Actions

Also available in: Atom PDF