Skip to content

Commit cd8082b

Browse files
author
Ludovic Vannoorenberghe
committed
add structured data tags (related to #41)
1 parent 1a13f29 commit cd8082b

File tree

1 file changed

+18
-16
lines changed

1 file changed

+18
-16
lines changed

views/package_version/show.ejs

+18-16
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,24 @@
11
<%- partial ('../shared/_navbar.ejs') %>
22
<% var pkgVersion = data; %>
3-
<section class="package">
3+
<section class="package" itemscope itemtype="http://schema.org/SoftwareApplication">
44
<div class="container">
55
<header>
6-
<h1><%- pkgVersion.package_name %></h1>
6+
<h1 itemprop="name"><%- pkgVersion.package_name %></h1>
77

88
<div class="select">
99
<select id="packageVersionSelect">
1010
<% for(var i = 0; i < pkgVersion.package.versions.length; i++) { %>
1111
<option data-uri="<%= pkgVersion.package.versions[i].uri %>"
1212
value="<%= pkgVersion.package.versions[i].version %>"
13-
<%= (pkgVersion.package.versions[i].version === pkgVersion.version) ? 'selected' : '' %> >
13+
<%= (pkgVersion.package.versions[i].version === pkgVersion.version) ? 'selected' : '' %>
14+
<%= (pkgVersion.package.versions[i].version === pkgVersion.version) ? 'itemprop="softwareVersion"' : '' %>
15+
>
1416
<%= pkgVersion.package.versions[i].version %>
1517
</option>
1618
<% } %>
1719
</select>
1820
</div>
19-
<div class="rating" title=<%= pkgVersion.rating %>>
20-
21+
<div itemprop="aggregateRating" itemscope itemtype="http://schema.org/AggregateRating" class="rating" title=<%= pkgVersion.rating %>>
2122
<% if (pkgVersion.rating) { %>
2223
<% for(var i=1; i <= 5; i++) { %>
2324
<i class="fa fa-star<%=
@@ -26,8 +27,9 @@
2627
: ""
2728
%>" aria-hidden="true"></i>
2829
<% } %>
30+
<span itemprop="ratingValue" hidden>(<%= pkgVersion.rating %></span>
2931
<% } %>
30-
<span>(<%= pkgVersion.reviews.length %> votes)</span>
32+
(<span itemprop="reviewCount"><%= pkgVersion.reviews.length %> </span>votes)
3133
</div>
3234
3335
@@ -46,7 +48,7 @@
4648
<dl>
4749
<% if (pkgVersion.maintainer) { %>
4850
<dt>Maintainer:</dt>
49-
<dd><a href="<%= pkgVersion.maintainer.uri %>">
51+
<dd><a itemprop="creator" href="<%= pkgVersion.maintainer.uri %>">
5052
<%= pkgVersion.maintainer.name %></a></dd>
5153
<% } %>
5254
<% if(pkgVersion.collaborators.length > 0) { %>
@@ -59,26 +61,26 @@
5961
<% } %>
6062
<% if (pkgVersion.release_date) { %>
6163
<dt>Release date:</dt>
62-
<dd><%= dateformat(pkgVersion.release_date, "mediumDate") %></dd>
64+
<dd itemprop="datePublished"><%= dateformat(pkgVersion.release_date, "mediumDate") %></dd>
6365
<% } %>
6466
<% if(pkgVersion.license) { %>
6567
<dt>License</dt>
66-
<dd><%= pkgVersion.license %></dd>
68+
<dd itemprop="license"><%= pkgVersion.license %></dd>
6769
<% } %>
6870
<% if(pkgVersion.copyright) { %>
6971
<dt>Copyright</dt>
70-
<dd><%= pkgVersion.copyright %></dd>
72+
<dd itemprop="copyrightHolder"><%= pkgVersion.copyright %></dd>
7173
<% } %>
7274
<% if(pkgVersion.url) { %>
7375
<dt>Package url</dt>
74-
<dd><%- pkgVersion.url %></dd>
76+
<dd itemprop="url"><%- pkgVersion.url %></dd>
7577
<% } %>
7678
<% if(pkgVersion.package.inViews && pkgVersion.package.inViews.length > 0) { %>
7779
<dt>In Views:</dt>
7880
<% var views = pkgVersion.package.inViews.map(function(view) {
7981
return '<a href="/taskviews/' + view.name + '">' + view.name + '</a>';
8082
}) -%>
81-
<dd><%- views.join(', ') %></dd>
83+
<dd itemprop="isPartOf"><%- views.join(', ') %></dd>
8284
<% } %>
8385
</dl>
8486
</div>
@@ -100,8 +102,8 @@
100102
</section>
101103
102104
<section class="package--info">
103-
<h4><%- pkgVersion.title %></h4>
104-
<p><%- pkgVersion.description %></p>
105+
<h4 itemprop="headline"><%- pkgVersion.title %></h4>
106+
<p itemprop="description"><%- pkgVersion.description %></p>
105107
</section>
106108
107109
<div id="chart" style="display: none" data-url="/api<%= path %>/downloads/per_day_last_month">
@@ -152,15 +154,15 @@
152154
<% if(!pkgVersion.url) {%>
153155
<%-md(pkgVersion.readmemd,"") %>
154156
<% }%>
155-
157+
156158
</section>
157159
<% } %>
158160
159161
<section class="package--dependencies">
160162
<h4>Dependencies</h4>
161163
<ul class="inline">
162164
<% for(var i = 0; i < pkgVersion.dependencies.length; i++) { %>
163-
<li><a href="<%= pkgVersion.dependencies[i].uri %>"><%= pkgVersion.dependencies[i].name %></a></li>
165+
<li ><a itemprop="softwareRequirements" href="<%= pkgVersion.dependencies[i].uri %>"><%= pkgVersion.dependencies[i].name %></a></li>
164166
<% } %>
165167
</ul>
166168
</section>

0 commit comments

Comments
 (0)