From 10b6dcebd0e70bb7680c321a4fd7062a9ef1a373 Mon Sep 17 00:00:00 2001 From: albert Date: Mon, 25 Feb 2013 09:43:17 -0500 Subject: [PATCH] fix rel attrs for pool links --- .../stylesheets/specific/comments.css.scss | 4 ++++ app/presenters/post_presenter.rb | 16 ++++++++++++---- 2 files changed, 16 insertions(+), 4 deletions(-) diff --git a/app/assets/stylesheets/specific/comments.css.scss b/app/assets/stylesheets/specific/comments.css.scss index de8d87a77..e6a1e85ea 100644 --- a/app/assets/stylesheets/specific/comments.css.scss +++ b/app/assets/stylesheets/specific/comments.css.scss @@ -1,6 +1,10 @@ @import "../common/000_vars.css.scss"; div.comments-for-post { + div.notices { + margin: 1em 0; + } + div.list-of-comments { article.comment { margin-bottom: 2em; diff --git a/app/presenters/post_presenter.rb b/app/presenters/post_presenter.rb index 9ebbf87ba..b40ffba36 100644 --- a/app/presenters/post_presenter.rb +++ b/app/presenters/post_presenter.rb @@ -93,7 +93,7 @@ class PostPresenter < Presenter if template.params[:pool_id].present? pool = Pool.where(:id => template.params[:pool_id]).first return if pool.nil? - html = pool_link_html(html, template, pool) + html = pool_link_html(html, template, pool, :include_rel => true) @post.pools.active.where("id <> ?", template.params[:pool_id]).each do |other_pool| html = pool_link_html(html, template, other_pool) @@ -108,19 +108,27 @@ class PostPresenter < Presenter html.join("\n").html_safe end - def pool_link_html(html, template, pool) + def pool_link_html(html, template, pool, options = {}) pool_html = ["
  • "] match_found = false + if options[:include_rel] + prev_rel = "prev" + next_rel = "next" + else + prev_rel = nil + next_rel = nil + end + if pool.neighbors(@post).previous - pool_html << template.link_to("«prev".html_safe, template.post_path(pool.neighbors(@post).previous, :pool_id => pool.id), :rel => "prev") + pool_html << template.link_to("«prev".html_safe, template.post_path(pool.neighbors(@post).previous, :pool_id => pool.id), :rel => prev_rel) match_found = true else pool_html << "«prev" end if pool.neighbors(@post).next - pool_html << template.link_to("next»".html_safe, template.post_path(pool.neighbors(@post).next, :pool_id => pool.id), :rel => "next") + pool_html << template.link_to("next»".html_safe, template.post_path(pool.neighbors(@post).next, :pool_id => pool.id), :rel => next_rel) match_found = true else pool_html << "next»"