Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Commit

Permalink
Add title attribute to user flair and linkflair selector span.
Browse files Browse the repository at this point in the history
This allows the full text to be discovered when the item is too long for the
entire span to be shown. This most often occurs in the flair selector.
  • Loading branch information
bboe authored and spladug committed Apr 30, 2012
1 parent 55f1149 commit 611892a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion r2/r2/templates/flairselectorlinksample.html
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
################################################################################

<%def name="flair()">
<span class="linkflairlabel">${thing.flair_text}</span>
<span class="linkflairlabel" title="${thing.flair_text}">${thing.flair_text}</span>
</%def>

<div class="linkflair ${' '.join('linkflair-%s' % c for c in thing.flair_css_class.split())}">
Expand Down
2 changes: 1 addition & 1 deletion r2/r2/templates/wrappeduser.html
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
<% enabled = user.flair_enabled %>
%endif
%if user.has_flair and enabled:
<span class="flair ${user.flair_css_class}">${user.flair_text}</span>
<span class="flair ${user.flair_css_class}" title="${user.flair_text}">${user.flair_text}</span>
%endif
</%def>

Expand Down

0 comments on commit 611892a

Please sign in to comment.