fu: Close-up of Fu, bringing a scoop of water to her mouth (Default)
fu ([personal profile] fu) wrote in [site community profile] changelog2011-02-08 12:58 pm

[dw-free] Enhancements to "find users with similar interests"

[commit: http://hg.dwscoalition.org/dw-free/rev/d3cafbea9bb9]

http://bugs.dwscoalition.org/show_bug.cgi?id=2018

Provide anchor links to the sections on the page.

Patch by [personal profile] kareila.

Files modified:
  • views/interests/findsim.tt
--------------------------------------------------------------------------------
diff -r 7acf6200abb8 -r d3cafbea9bb9 views/interests/findsim.tt
--- a/views/interests/findsim.tt	Tue Feb 08 20:54:56 2011 +0800
+++ b/views/interests/findsim.tt	Tue Feb 08 20:57:40 2011 +0800
@@ -14,6 +14,7 @@
 [%- sections.head = BLOCK %]
 <style type='text/css'>
     .accountlist { padding-bottom: 3em; }
+    .anchorlinks { text-align: center; margin: 2em 0; }
 </style>
 [% END -%]
 
@@ -33,9 +34,23 @@
 [%- END -%]
 <p>[% '.text' | ml(user = findsim_u.ljuser_display) %]</p>
 
+<div class="anchorlinks">
+[%- IF findsim_count.P -%]
+    <a href="#personal">[% '.accounts.person' | ml %]</a>
+    [%- IF findsim_count.C || findsim_count.I; '&nbsp;|&nbsp;'; END -%]
+[%- END -%]
+[%- IF findsim_count.C -%]
+    <a href="#community">[% '.accounts.comm' | ml %]</a>
+    [%- IF findsim_count.I; '&nbsp;|&nbsp;'; END -%]
+[%- END -%]
+[%- IF findsim_count.I -%]
+    <a href="#identity">[% '.accounts.id' | ml %]</a>
+[%- END -%]
+</div>
+
 [%- IF findsim_count.P -%]
 <div class="accountlist">
-<h2>[% '.accounts.person' | ml %]</h2>
+<a name="personal"><h2>[% '.accounts.person' | ml %]</h2></a>
 <table cellpadding='3'><thead><tr valign='bottom'>
     <th>#</th>
     <th width='250'>[% 'username' | ml %]</th>
@@ -53,7 +68,7 @@
 
 [%- IF findsim_count.C -%]
 <div class="accountlist">
-<h2>[% '.accounts.comm' | ml %]</h2>
+<a name="community"><h2>[% '.accounts.comm' | ml %]</h2></a>
 <table cellpadding='3'><thead><tr valign='bottom'>
     <th>#</th>
     <th width='250'>[% 'username' | ml %]</th>
@@ -71,7 +86,7 @@
 
 [%- IF findsim_count.I -%]
 <div class="accountlist">
-<h2>[% '.accounts.id' | ml %]</h2>
+<a name="identity"><h2>[% '.accounts.id' | ml %]</h2></a>
 <table cellpadding='3'><thead><tr valign='bottom'>
     <th>#</th>
     <th width='250'>[% 'username' | ml %]</th>
--------------------------------------------------------------------------------