[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#532535: packages.debian.org: issues with version anchor



Package: www.debian.org
Severity: minor
Tag: patch

Hi,

The version anchors are quite strange (on top on being hidden) :
http://packages.debian.org/changelogs/pool/main/v/vidalia/vidalia_0.1.13-1/changelog#versionversion0.1.9-1

In conf/tmpl/content.tmpl, there are two variables providing the
version in different ways, CONTENT_VERSION_ID and CONTENT_VERSION.

I don't know whether it's better to use :
<TMPL_VARNAME="CONTENT_VERSION_ID">
or version<TMPL_VAR NAME="CONTENT_VERSION">

By the way, that would be nice to make the anchor visible, under the
version.

Here is a little patch for this.

-- 
Simon Paillard
diff --git a/conf/tmpl/content.tmpl b/conf/tmpl/content.tmpl
index 693b9d4..0c85e16 100644
--- a/conf/tmpl/content.tmpl
+++ b/conf/tmpl/content.tmpl
@@ -4,9 +4,9 @@
 <TMPL_VAR NAME="CONTENT_YEAR">
 </h2>
 <TMPL_LOOP NAME="CONTENT_VERSIONS">
-<h3 class="entry_header" id="version<TMPL_VAR NAME="CONTENT_VERSION_ID">">
+<h3 class="entry_header" id="<TMPL_VAR NAME="CONTENT_VERSION_ID">">
 	<a class="packagelink" href="http://packages.debian.org/src:<TMPL_VAR ESCAPE="URL" NAME="CONTENT_SOURCE">"><TMPL_VAR NAME="CONTENT_SOURCE"></a>
-	 (<TMPL_VAR NAME="CONTENT_VERSION">)
+	 (<a href="<TMPL_VAR NAME="CONTENT_VERSION_ID">"><TMPL_VAR NAME="CONTENT_VERSION"></a>)
 	<span class="<TMPL_VAR NAME="CONTENT_DISTRIBUTION_NORM">"><TMPL_VAR NAME="CONTENT_DISTRIBUTION"></span>;
 	urgency=<span class="<TMPL_VAR NAME="CONTENT_URGENCY_NORM">"><TMPL_VAR NAME="CONTENT_URGENCY"></span>
 </h3>

Reply to: