Changesets can be listed by changeset number.
The Git repository is here.
Changeset 188
Order topics by display name, not psuedo-random login ID (Hub artefact).
Ensure 'author' fields in RSS feeds also use display name, not login ID.
Standardised titles for RSS feeds across all applications. Tidied up
topic lists, improving base CSS and simplifying document layout.
- Comitted by: rool
- Date: Wednesday April 04 18:24:44 2007 (over 17 years ago)
Affected files:
- rool/rails/beast/trunk/app/models/topic.rb (diff)
- rool/rails/beast/trunk/app/views/forums/show.rhtml (diff)
- rool/rails/beast/trunk/app/views/layouts/_post.rxml (diff)
- rool/rails/beast/trunk/app/views/posts/index.rxml (diff)
- rool/rails/beast/trunk/app/views/posts/monitored.rxml (diff)
- rool/rails/beast/trunk/app/views/topics/show.rxml (diff)
- rool/rails/beast/trunk/public/stylesheets/display.css (diff)
rool/rails/beast/trunk/app/models/topic.rb:
prev. | current | |
belongs_to :forum, :counter_cache => true | ||
belongs_to :user | ||
has_many :monitorships | ||
5 | | |
5 | has_many :monitors, :through => :monitorships, :conditions => ['monitorships.active = ?', true], :source => :user, :order => 'users.display_name' | |
has_many :posts, :order => 'posts.created_at', :dependent => :destroy do | ||
def last |
rool/rails/beast/trunk/app/views/forums/show.rhtml:
prev. | current | |
<%= pluralize @forum.topics_count, 'topic' %>, <%= pluralize @forum.posts_count, 'post' %> | ||
</p> | ||
39 | ||
40 | ||
<% if @topic_pages.page_count > 1 -%> | ||
42 | ||
43 | ||
40 | <p> | |
41 | <%= link_to 'New topic', new_topic_path(@forum) %> | | |
42 | Pages: <strong><%= pagination_links @topic_pages, :window_size => 10 %></strong> | |
43 | </p> | |
<% end -%> | ||
<table border="0" cellspacing="0" cellpadding="0" class="wide topics"> | ||
... | ... | |
<% end %> | ||
</table> | ||
90 | <p> | |
91 | <%= link_to 'New topic', new_topic_path(@forum) %> | |
92 | <% if @topic_pages.page_count > 1 -%> | |
93 | | Pages: <strong><%= pagination_links @topic_pages, :window_size => 10 %></strong> | |
94 | <% end -%> | |
<% if @topic_pages and @topic_pages.current.next %> | ||
91 | ||
96 | | <%= link_to "Next page", { :page => @topic_pages.current.next }.merge(params.reject{|k,v| k=="page"}) %> | |
<% end %> | ||
93 | ||
94 | ||
95 | ||
96 | ||
97 | ||
98 | ||
98 | </p> |
rool/rails/beast/trunk/app/views/layouts/_post.rxml:
prev. | current | |
xm.description post.body_html | ||
xm.pubDate post.created_at.rfc822 | ||
xm.guid [request.host_with_port, post.forum_id.to_s, post.topic_id.to_s, post.id.to_s].join(":"), "isPermaLink" => "false" | ||
6 | | |
6 | xm.author "#{post.user.display_name}" | |
xm.link topic_url(post.forum_id, post.topic_id) | ||
end |
rool/rails/beast/trunk/app/views/posts/index.rxml:
prev. | current | |
'xmlns:opensearch' => "http://a9.com/-/spec/opensearch/1.1/", | ||
'xmlns:atom' => "http://www.w3.org/2005/Atom" do | ||
xml.channel do | ||
7 | | |
7 | xml.title "ROOL Forum: #{search_posts_title}" | |
xml.link "http://#{request.host_with_port}#{search_posts_path}" | ||
xml.language "en-us" | ||
xml.ttl "60" |
rool/rails/beast/trunk/app/views/posts/monitored.rxml:
prev. | current | |
'xmlns:opensearch' => "http://a9.com/-/spec/opensearch/1.1/", | ||
'xmlns:atom' => "http://www.w3.org/2005/Atom" do | ||
xml.channel do | ||
7 | | |
7 | xml.title "ROOL Forum: Posts that #{@user.display_name} is monitoring" | |
xml.link monitored_posts_url(@user) | ||
xml.language "en-us" | ||
xml.ttl "60" |
rool/rails/beast/trunk/app/views/topics/show.rxml:
prev. | current | |
'xmlns:opensearch' => "http://a9.com/-/spec/opensearch/1.1/", | ||
'xmlns:atom' => "http://www.w3.org/2005/Atom" do | ||
xml.channel do | ||
7 | | |
7 | xml.title "ROOL Forum: Recent Posts in '#{@topic.title}'" | |
xml.link topic_url(@forum, @topic) | ||
xml.language "en-us" | ||
xml.ttl "60" |
rool/rails/beast/trunk/public/stylesheets/display.css:
prev. | current | |
#content p.pages | ||
{ | ||
32 | | |
font-size:0.85em; | ||
margin-top:1.2em; | ||
} | ||
... | ... | |
#content a.utility | ||
{ | ||
font-weight:normal; | ||
196 | | |
font-size:0.8em; | ||
} | ||
#content a.remove { color:#c00; } | ||
201 | ||
202 | ||
203 | | |
204 | ||
#content a.subtle { color:#369; } | ||
#content a.subtle:hover { color:blue; } | ||
... | ... | |
{ | ||
border-collapse:collapse; | ||
} | ||
376 | ||
#content table tr th | ||
{ | ||
background:#ddd; |