diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb index 258ad2e4..a633727e 100644 --- a/app/helpers/application_helper.rb +++ b/app/helpers/application_helper.rb @@ -160,7 +160,7 @@ def tag_link(type, tag, link_class=nil) when :venue then venues_path(:tag => tag.name) end - link_classes = [link_class] + link_classes = [link_class, "p-category"] link_classes << "external #{tag.machine_tag[:namespace]} #{tag.machine_tag[:predicate]}" if tag.machine_tag[:url] link_to escape_once(tag.name), (tag.machine_tag[:url] || internal_url), :class => link_classes.compact.join(' ') diff --git a/app/helpers/time_range_helper.rb b/app/helpers/time_range_helper.rb index f166b378..f0a5e7ea 100644 --- a/app/helpers/time_range_helper.rb +++ b/app/helpers/time_range_helper.rb @@ -89,14 +89,14 @@ def to_s # Combine the pieces results = [] - results << %Q|| if @format == :hcal + results << %Q|| if @format == :hcal + results << %Q|| if @format == :hcal if end_format_list results << conjunction - results << %Q|| if @format == :hcal + results << %Q|| if @format == :hcal + results << %Q|| if @format == :hcal end results.join('').html_safe end diff --git a/app/views/events/_item.html.erb b/app/views/events/_item.html.erb index 5a7c3d76..ed9e9387 100644 --- a/app/views/events/_item.html.erb +++ b/app/views/events/_item.html.erb @@ -9,35 +9,35 @@ html_classes = "single_event" html_classes << " contentbar" if has_contentbar %>
-
-

<%= event.title %>

+
+

<%= event.title %>

<% if event.end_time %> <% end %>
<%= normalize_time(event) -%>
<% unless event.venue.blank? %> -
" itemprop="location" itemscope itemtype="http://schema.org/Place"> +
" itemprop="location" itemscope itemtype="http://schema.org/Place"> <%= google_map event %> - - <%= event.venue.title -%> + + <%= event.venue.title -%> <% if event.venue.closed? %>

This venue is no longer open for business.

<% end %> -
+
<% if !event.venue.street_address.blank? -%> -
<%= event.venue.street_address -%>
+
<%= event.venue.street_address -%>
<% end %> <% if !event.venue.locality.blank? -%> - <%= event.venue.locality -%><%= "," if event.venue.region.present? or event.venue.postal_code.present? or event.venue.country.present? %> + <%= event.venue.locality -%><%= "," if event.venue.region.present? or event.venue.postal_code.present? or event.venue.country.present? %> <% end -%> <% if !event.venue.region.blank? -%> - <%= event.venue.region -%><%= "," if event.venue.country.present? and not event.venue.postal_code.present? %> + <%= event.venue.region -%><%= "," if event.venue.country.present? and not event.venue.postal_code.present? %> <% end -%> <% if !event.venue.postal_code.blank? -%> - <%= event.venue.postal_code -%><%= "," if event.venue.country.present? %> + <%= event.venue.postal_code -%><%= "," if event.venue.country.present? %> <% end -%> <% if !event.venue.country.blank? -%> - <%= event.venue.country -%> + <%= event.venue.country -%> <% end -%> <% if event.venue && event.venue.has_full_address? -%> (map) @@ -55,7 +55,7 @@ html_classes << " contentbar" if has_contentbar <% end -%> <% unless event.venue_details.blank? -%> -

+

<%= format_description(event.venue_details) %>

<% end -%> @@ -67,14 +67,14 @@ html_classes << " contentbar" if has_contentbar <% unless event.url.blank? -%>

Website

- <%= link_to "#{truncate(event.url, :length => 128)}", h(event.url), :class => "url", :rel => "nofollow", + <%= link_to "#{truncate(event.url, :length => 128)}", h(event.url), :class => "url u-url", :rel => "nofollow", :itemprop => "url" %>
<% end -%> <% unless event.description.blank? -%>

Description

-
+
<%= format_description(event.description) %>
<% end -%> diff --git a/app/views/events/_list_item.html.erb b/app/views/events/_list_item.html.erb index 8bbd45cd..c4c2157d 100644 --- a/app/views/events/_list_item.html.erb +++ b/app/views/events/_list_item.html.erb @@ -4,7 +4,7 @@ opts ||= {} opts[:dates] = opts.has_key?(:dates) ? opts[:dates] : true show_year ||= false %> -
+
<% date_class = (opts[:dates]==false || !new_day) ? ' hidden' : '' -%>
<%= today_tomorrow_or_weekday(event) %>
@@ -14,20 +14,20 @@ show_year ||= false <% end %>
- <%= event.title %> + <%= event.title %>
<%= normalize_time(event, :context => event.start_time.to_date) -%> <% if event.venue && !event.venue.title.blank? %> -
<%= event.venue.title -%> +
<%= event.venue.title -%> <% end -%>
<% if !event.description.blank? %> -
+
<%= format_description(event.description) -%>
<% end -%> <% if !event.url.blank? -%> - <%= link_to "Website", h(event.url), :class => "url", :rel => "nofollow" -%> + <%= link_to "Website", h(event.url), :class => "url u-url", :rel => "nofollow" -%> <% end -%>
diff --git a/app/views/events/_table.html.erb b/app/views/events/_table.html.erb index 2b3f43c4..94ac998b 100644 --- a/app/views/events/_table.html.erb +++ b/app/views/events/_table.html.erb @@ -31,7 +31,7 @@ rowspans = calculate_rowspans(events) <% unless events.size==0 %> <% events.each_with_index do |event, index| %> - + <% if rowspans[index] > 0 %>
@@ -42,18 +42,18 @@ rowspans = calculate_rowspans(events) <% end %> - <%= event.title %>
+ <%= event.title %>
<%= normalize_time(event, :context => event.start_time.to_date) -%> <% if event.venue && !event.venue.title.blank? %> - <%= event.venue.title -%> + <%= event.venue.title -%> <% end -%> <% if !event.description.blank? %> -
+
<%= format_description(event.description) -%>
<% end -%> <% if !event.url.blank? -%> - <%= link_to "Website", event.url, :class => "url", :rel => "nofollow" -%> + <%= link_to "Website", event.url, :class => "url u-url", :rel => "nofollow" -%> <% end -%> diff --git a/spec/helpers/time_range_helper_spec.rb b/spec/helpers/time_range_helper_spec.rb index b68e9bbd..3148acc5 100644 --- a/spec/helpers/time_range_helper_spec.rb +++ b/spec/helpers/time_range_helper_spec.rb @@ -12,17 +12,17 @@ tests = [ # comment, end_time, results_without_context, results_with_context [ "start time only", nil, - "Tuesday, April 1, 2008 at 9am", - "9am"], + "", + ""], [ "same day & am-pm", DateTime.new(2008, 4, 1, 11, 00), - "Tuesday, April 1, 2008 from 911am", - "911am" ], + "", + "" ], [ "same day, different am-pm", DateTime.new(2008, 4, 1, 13, 30), - "Tuesday, April 1, 2008 from 9am1:30pm", - "9am1:30pm" ], + "", + "" ], [ "different days", DateTime.new(2009, 4, 1, 13, 30), - "Tuesday, April 1, 2008 at 9am through Wednesday, April 1, 2009 at 1:30pm", - "9am through Wednesday, April 1, 2009 at 1:30pm" ] + " through ", + " through " ] ] [nil, Date.new(2008, 4, 1)].each do |context_date|