Skip to content

Commit e42d39e

Browse files
authored
Merge pull request #333 from canjs/331-wrap-results-in-a-tag
Wrap search results in an anchor tag. Closes #331
2 parents c349707 + 5c8fd02 commit e42d39e

File tree

2 files changed

+15
-11
lines changed

2 files changed

+15
-11
lines changed

static/search.less

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,9 @@
8484
a{
8585
color: @link-color;
8686
padding-bottom:0;
87+
&:hover {
88+
text-decoration: none;
89+
}
8790
}
8891
.name{
8992
line-height: 1;

templates/search-results.stache

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -18,19 +18,20 @@
1818
<li>
1919
<a href="{{docUrl}}" title="{{result.name}}" class="result-name">
2020
{{#if result.title}}{{result.title}}{{else}}{{result.name}}{{/if}}
21-
</a>
21+
2222

23-
{{#if result.title}}
24-
<span class="name">
25-
({{result.name}})
26-
</span>
27-
{{/if}}
23+
{{#if result.title}}
24+
<span class="name">
25+
({{result.name}})
26+
</span>
27+
{{/if}}
2828

29-
{{#if result.description}}
30-
<div class="result-description" title="{{result.description}}">
31-
{{result.description}}
32-
</div>
33-
{{/if}}
29+
{{#if result.description}}
30+
<div class="result-description" title="{{result.description}}">
31+
{{result.description}}
32+
</div>
33+
{{/if}}
34+
</a>
3435
</li>
3536
{{/each}}
3637
</ul>

0 commit comments

Comments
 (0)