Layout of activities - not ready

Caio Almeida · Daniela Feitosa
1 parent 71878155
Showing 7 changed files with 52 additions and 28 deletions Show diff stats
app/models/article.rb
app/views/profile/_comment.rhtml
app/views/profile/_create_article.rhtml
app/views/profile/_profile_activity.rhtml
config/environments/development.rb
public/stylesheets/application.css
test/unit/article_test.rb
@@ -2,7 +2,7 @@ require &#39;hpricot&#39;
 class Article < ActiveRecord::Base
-  track_actions :create_article, :after_create, :keep_params => [:name, :url, :lead], :if => Proc.new { |a| a.is_trackable? && !a.image? }
+  track_actions :create_article, :after_create, :keep_params => [:name, :url, :lead, :first_image], :if => Proc.new { |a| a.is_trackable? && !a.image? }
   # xss_terminate plugin can't sanitize array fields
   before_save :sanitize_tag_list
@@ -566,6 +566,11 @@ class Article &lt; ActiveRecord::Base
     ActionTracker::Record.find_by_target_type_and_target_id 'Article', self.id
   end
+  def first_image
+    img = Hpricot(self.lead.to_s).search('img[@src]').first || Hpricot(self.body.to_s).search('img').first
+    img.nil? ? '' : img.attributes['src']
+  end
+
   private
   def sanitize_tag_list
@@ -4,8 +4,7 @@
   <div class="comment-content comment-logged-in">
   <% if Person.find(comment.author_id) %>
-    <%= link_to image_tag(profile_icon(Person.find(comment.author_id), :minor)) +
-                content_tag('span', Person.find(comment.author_id).name, :class => 'comment-info'),
+    <%= link_to image_tag(profile_icon(Person.find(comment.author_id), :minor)),
         Person.find(comment.author_id).url,
         :class => 'comment-picture',
         :title => Person.find(comment.author_id).name
@@ -13,10 +12,8 @@
   <% end %>
   <div class="comment-details">
-    <h4><%= comment.title %></h4>
     <div class="comment-text">
-      <p/>
-      <%= txt2html comment.body %>
+      <%= link_to(Person.find(comment.author_id).name, Person.find(comment.author_id).url) %><br /><%= txt2html comment.body %>
     </div>
     <div class="profile-activity-time">
       <%= time_ago_as_sentence(comment.created_at) %>
@@ -3,7 +3,7 @@
 </div>
 <div class='profile-activity-description'>
   <p class='profile-activity-text'><%= link_to activity.user.name, activity.user.url %> <%= describe activity %></p>
-  <p class='profile-activity-text'><%= activity.params['lead'] %></p>
+  <div class='profile-activity-lead'><%= image_tag(activity.params['first_image']) if activity.params['first_image'] %><%= strip_tags(activity.params['lead']).gsub(/(\xA0|\xC2|\s)+/, ' ').gsub(/^\s+/, '') %></div>
   <p class='profile-activity-time'><%= time_ago_as_sentence(activity.created_at) + ' ' + _('ago') %></p>
   <div class='profile-wall-actions'>
     <%= link_to_remote(content_tag(:span, _('Remove')), :url =>{:action => 'remove_activity', :activity_id => activity.id}, :update => "profile-activity-item-#{activity.id}") if logged_in? && current_person == @profile %>
@@ -2,10 +2,6 @@
   <%= render :partial => activity.verb, :locals => { :activity => activity }%>
   <hr />
-  <% if logged_in? && current_person.follows?(activity.user) %>
-    <span class='profile-wall-send-reply'><%= link_to_function _('Comment'), "hide_and_show(['#profile-wall-reply-response-#{activity.id}'],['#profile-wall-reply-#{activity.id}', '#profile-wall-reply-form-#{activity.id}']);$('reply_content_#{activity.id}').value='';$('reply_content_#{activity.id}').focus();$('activity_id_#{activity.id}').value='#{activity.id}';return false", :class => "profile-send-reply" %></span>
-  <% end %>
-
   <ul class="profile-wall-activities-comments" style='padding-left: 50px;width:auto'>
     <%= render :partial => 'comment', :collection => activity.comments %>
   </ul>
 # Settings specified here will take precedence over those in config/environment.rb
-# The production environment is meant for finished, "live" apps.
-# Code is not reloaded between requests
-config.cache_classes = true
+# In the development environment your application's code is reloaded on
+# every request.  This slows down response time but is perfect for development
+# since you don't have to restart the webserver when you make code changes.
+config.cache_classes = false
-# Use a different logger for distributed setups
-# config.logger = SyslogLogger.new
+# Log error messages when you accidentally call methods on nil.
+config.whiny_nils = true
-# Full error reports are disabled and caching is turned on
-config.action_controller.consider_all_requests_local = false
-config.action_controller.perform_caching             = true
-config.action_view.cache_template_loading            = true
+# Show full error reports and disable caching
+config.action_controller.consider_all_requests_local = true
+config.action_view.debug_rjs                         = true
+config.action_controller.perform_caching             = false
-# Enable serving of images, stylesheets, and javascripts from an asset server
-# config.action_controller.asset_host                  = "http://assets.example.com"
-
-# Disable delivery errors, bad email addresses will be ignored
-# config.action_mailer.raise_delivery_errors = false
-
-config.cache_store = :mem_cache_store, "localhost"
+# Don't care if the mailer can't send
+config.action_mailer.raise_delivery_errors = false
@@ -5883,11 +5883,24 @@ h1#agenda-title {
 #profile-activity li, #profile-network li, #profile-wall li {
   display: block;
   padding: 10px;
-  margin-bottom: 0;
+  margin-bottom: 10px;
   background-color: #f0f0f1;
   border-bottom: 0;
 }
+.profile-activity-lead img {
+  width: 64px;
+  float: left;
+  margin-right: 5px;
+}
+
+.profile-activity-lead {
+  width: 370px;
+  float: right;
+  text-align: left;
+  margin: 5px 0;
+}
+
 #profile-activity li a, #profile-network li a, #profile-wall li a,
 #profile-wall .profile-wall-send-reply {
   color: #333;
@@ -6169,6 +6182,7 @@ h1#agenda-title {
   background: #e8e8e8;
   border-bottom: 1px solid #d2d2d2 !important;
   border-top: 1px solid #fff;
+  margin-bottom: 0;
 }
 #profile-wall .profile-wall-activities-comments img {
@@ -1584,4 +1584,20 @@ assert_equal &#39;bla&#39;, profile.articles.map(&amp;:comments_count)
     assert_includes Article.created_between(from, to), article2
     assert_not_includes Article.created_between(from, to), article3
   end
+
+  should 'get first image from lead' do
+    a = fast_create(Article, :body => '<p>Foo</p><p><img src="bar.png" />Bar<img src="foo.png" /></p>',
+                             :abstract => '<p>Lead</p><p><img src="leadbar.png" />Bar<img src="leadfoo.png" /></p>')
+    assert_equal 'leadbar.png', a.first_image
+  end
+
+  should 'get first image from body' do
+    a = fast_create(Article, :body => '<p>Foo</p><p><img src="bar.png" />Bar<img src="foo.png" /></p>')
+    assert_equal 'bar.png', a.first_image
+  end
+
+  should 'not get first image from anywhere' do
+    a = fast_create(Article, :body => '<p>Foo</p><p>Bar</p>')
+    assert_equal '', a.first_image
+  end
 end
	@@ -3,7 +3,7 @@		@@ -3,7 +3,7 @@
3	</div>	3	</div>
4	<div class='profile-activity-description'>	4	<div class='profile-activity-description'>
5	<p class='profile-activity-text'><%= link_to activity.user.name, activity.user.url %> <%= describe activity %></p>	5	<p class='profile-activity-text'><%= link_to activity.user.name, activity.user.url %> <%= describe activity %></p>
6	- <p class='profile-activity-text'><%= activity.params['lead'] %></p>	6	+ <div class='profile-activity-lead'><%= image_tag(activity.params['first_image']) if activity.params['first_image'] %><%= strip_tags(activity.params['lead']).gsub(/(\xA0\|\xC2\|\s)+/, ' ').gsub(/^\s+/, '') %></div>
7	<p class='profile-activity-time'><%= time_ago_as_sentence(activity.created_at) + ' ' + _('ago') %></p>	7	<p class='profile-activity-time'><%= time_ago_as_sentence(activity.created_at) + ' ' + _('ago') %></p>
8	<div class='profile-wall-actions'>	8	<div class='profile-wall-actions'>
9	<%= link_to_remote(content_tag(:span, _('Remove')), :url =>{:action => 'remove_activity', :activity_id => activity.id}, :update => "profile-activity-item-#{activity.id}") if logged_in? && current_person == @profile %>	9	<%= link_to_remote(content_tag(:span, _('Remove')), :url =>{:action => 'remove_activity', :activity_id => activity.id}, :update => "profile-activity-item-#{activity.id}") if logged_in? && current_person == @profile %>
1	# Settings specified here will take precedence over those in config/environment.rb	1	# Settings specified here will take precedence over those in config/environment.rb
2		2
3	-# The production environment is meant for finished, "live" apps.
4	-# Code is not reloaded between requests
5	-config.cache_classes = true	3	+# In the development environment your application's code is reloaded on
		4	+# every request. This slows down response time but is perfect for development
		5	+# since you don't have to restart the webserver when you make code changes.
		6	+config.cache_classes = false
6		7
7	-# Use a different logger for distributed setups
8	-# config.logger = SyslogLogger.new	8	+# Log error messages when you accidentally call methods on nil.
		9	+config.whiny_nils = true
9		10
10	-# Full error reports are disabled and caching is turned on
11	-config.action_controller.consider_all_requests_local = false
12	-config.action_controller.perform_caching = true
13	-config.action_view.cache_template_loading = true	11	+# Show full error reports and disable caching
		12	+config.action_controller.consider_all_requests_local = true
		13	+config.action_view.debug_rjs = true
		14	+config.action_controller.perform_caching = false
14		15
15	-# Enable serving of images, stylesheets, and javascripts from an asset server
16	-# config.action_controller.asset_host = "http://assets.example.com"
17	-
18	-# Disable delivery errors, bad email addresses will be ignored
19	-# config.action_mailer.raise_delivery_errors = false
20	-
21	-config.cache_store = :mem_cache_store, "localhost"	16	+# Don't care if the mailer can't send
		17	+config.action_mailer.raise_delivery_errors = false
	@@ -1584,4 +1584,20 @@ assert_equal 'bla', profile.articles.map(&:comments_count)		@@ -1584,4 +1584,20 @@ assert_equal 'bla', profile.articles.map(&:comments_count)
1584	assert_includes Article.created_between(from, to), article2	1584	assert_includes Article.created_between(from, to), article2
1585	assert_not_includes Article.created_between(from, to), article3	1585	assert_not_includes Article.created_between(from, to), article3
1586	end	1586	end
		1587	+
		1588	+ should 'get first image from lead' do
		1589	+ a = fast_create(Article, :body => '<p>Foo</p><p><img src="bar.png" />Bar<img src="foo.png" /></p>',
		1590	+ :abstract => '<p>Lead</p><p><img src="leadbar.png" />Bar<img src="leadfoo.png" /></p>')
		1591	+ assert_equal 'leadbar.png', a.first_image
		1592	+ end
		1593	+
		1594	+ should 'get first image from body' do
		1595	+ a = fast_create(Article, :body => '<p>Foo</p><p><img src="bar.png" />Bar<img src="foo.png" /></p>')
		1596	+ assert_equal 'bar.png', a.first_image
		1597	+ end
		1598	+
		1599	+ should 'not get first image from anywhere' do
		1600	+ a = fast_create(Article, :body => '<p>Foo</p><p>Bar</p>')
		1601	+ assert_equal '', a.first_image
		1602	+ end
1587	end	1603	end