[myexperiment-hackers] [2769] trunk/app/views/users/_listing.rhtml: Tidy up separator between website/email and field/occupation.

[noreply]

Revision

2769

Author

dtm

Date

2011-10-28 06:44:31 -0400 (Fri, 28 Oct 2011)

Log Message

Tidy up separator between website/email and field/occupation.
Add rel="nofollow" to website link.

Modified Paths

• trunk/app/views/users/_listing.rhtml

Diff

Modified: trunk/app/views/users/_listing.rhtml (2768 => 2769)

--- trunk/app/views/users/_listing.rhtml	2011-10-27 12:56:26 UTC (rev 2768)
+++ trunk/app/views/users/_listing.rhtml	2011-10-28 10:44:31 UTC (rev 2769)
@@ -34,27 +34,31 @@
           
     <% if user.profile %>
           
+     <% separator ="" %>
       <p style="font-size: 85%;">
         <% unless user.profile.website.blank? %>
           <b>Website:</b>
-          <%= link_to h(user.profile.website), h(user.profile.website), :popup => true %>
-          |
+          <%= link_to h(user.profile.website), h(user.profile.website), :rel => "nofollow", :popup => true %>
+          <% separator = "|" %>
         <% end %>
               
         <% unless user.profile.email.blank? %>
+          <%= separator %>
           <b>Email (public):</b>
           <%= mail_to user.profile.email, nil, {  :encode => "hex", :replace_at => " [at] " } %>
         <% end %>
       </p>
               
+     <% separator ="" %>
       <p style="font-size: 85%;">
         <% unless user.profile.field_or_industry.blank? %>
           <b>Field/Industry:</b>
           <%= h user.profile.field_or_industry %>
-          |
+          <% separator = "|" %>
         <% end %>
               
         <% unless user.profile.occupation_or_roles.blank? %>
+          <%= separator %>
           <b>Occupation/Role(s):</b>
           <%= h user.profile.occupation_or_roles %>
         <% end %>