Set og:description meta tag to diary entry description
This commit is contained in:
parent
5573df7741
commit
5f4ae35814
3 changed files with 25 additions and 2 deletions
|
@ -70,7 +70,8 @@ class DiaryEntriesController < ApplicationController
|
|||
@title = t ".title", :user => params[:display_name], :title => @entry.title
|
||||
@opengraph_properties = {
|
||||
"og:image" => @entry.body.image,
|
||||
"og:image:alt" => @entry.body.image_alt
|
||||
"og:image:alt" => @entry.body.image_alt,
|
||||
"og:description" => @entry.body.description
|
||||
}
|
||||
@comments = can?(:unhide, DiaryComment) ? @entry.comments : @entry.visible_comments
|
||||
else
|
||||
|
|
|
@ -7,7 +7,7 @@ module OpenGraphHelper
|
|||
"og:title" => title || t("layouts.project_name.title"),
|
||||
"og:type" => "website",
|
||||
"og:url" => url_for(:only_path => false),
|
||||
"og:description" => t("layouts.intro_text")
|
||||
"og:description" => properties["og:description"] || t("layouts.intro_text")
|
||||
}.merge(
|
||||
opengraph_image_properties(properties)
|
||||
)
|
||||
|
|
|
@ -744,6 +744,28 @@ class DiaryEntriesControllerTest < ActionDispatch::IntegrationTest
|
|||
assert_dom "head meta[property='og:image:alt']", :count => 0
|
||||
end
|
||||
|
||||
def test_show_no_og_description
|
||||
user = create(:user)
|
||||
diary_entry = create(:diary_entry, :user => user, :body => "")
|
||||
|
||||
get diary_entry_path(user, diary_entry)
|
||||
assert_response :success
|
||||
assert_dom "head meta[property='og:description']" do
|
||||
assert_dom "> @content", I18n.t("layouts.intro_text")
|
||||
end
|
||||
end
|
||||
|
||||
def test_show_og_description
|
||||
user = create(:user)
|
||||
diary_entry = create(:diary_entry, :user => user, :body => "# Hello\n\n\n\nFirst paragraph.\n\nSecond paragraph.")
|
||||
|
||||
get diary_entry_path(user, diary_entry)
|
||||
assert_response :success
|
||||
assert_dom "head meta[property='og:description']" do
|
||||
assert_dom "> @content", "First paragraph."
|
||||
end
|
||||
end
|
||||
|
||||
def test_hide
|
||||
user = create(:user)
|
||||
diary_entry = create(:diary_entry, :user => user)
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue