Patch #2607
closed
Avatars display in forums
90%
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
Related issues
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 {
Updated by Brad Beattie almost 16 years ago
Hrm. On one-line posts, this can cause the subsequent message h4 to indent incorrectly.
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.
Updated by Brad Beattie almost 16 years ago
- File application.css application.css added
- File show.rhtml show.rhtml added
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.
Updated by Brad Beattie almost 16 years ago
- File application.css application.css added
Minor clear: left fix.
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.
Updated by Brad Beattie almost 16 years ago
- File avatars.patch avatars.patch added
Right you are, good sir. When I made this ticket I was still getting into the swing of things.
Updated by Oshyan Greene almost 15 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!
Updated by Felix Schäfer almost 15 years ago
- Status changed from New to Closed