diff options
| author | Paul Arthur <paul.arthur@flowerysong.com> | 2012-09-20 11:44:42 -0400 |
|---|---|---|
| committer | Paul Arthur <paul.arthur@flowerysong.com> | 2012-09-20 11:44:42 -0400 |
| commit | 7ff92215def3ad11b90f517461fa7db440a1218f (patch) | |
| tree | bca6422b603522b8e97258cec81826d2868d0b22 /templates | |
| parent | bd7bf80aae0e8613da2fd60812167e06dd5ce9d5 (diff) | |
| download | ampache-7ff92215def3ad11b90f517461fa7db440a1218f.tar.gz ampache-7ff92215def3ad11b90f517461fa7db440a1218f.tar.bz2 ampache-7ff92215def3ad11b90f517461fa7db440a1218f.zip | |
FS#142 - Clicking on song results in downloading
Change the title link in a song row so it links to the song details.
While there's no real value in obfuscating the fact that a stream
can be downloaded, there is value in a consistent interface. The
song title being a stream link is inconsistent with other parts of
the row (such as album and artist), inconsistent with other times we
display songs (such as Recently Played), and doesn't consistently match
the currently selected playback method.
Diffstat (limited to 'templates')
| -rw-r--r-- | templates/show_song_row.inc.php | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/templates/show_song_row.inc.php b/templates/show_song_row.inc.php index 5913e64a..2e55076d 100644 --- a/templates/show_song_row.inc.php +++ b/templates/show_song_row.inc.php @@ -30,7 +30,7 @@ <td class="cel_add"> <?php echo Ajax::button('?action=basket&type=song&id=' . $song->id,'add', T_('Add'),'add_' . $song->id); ?> </td> -<td class="cel_song"><a href="<?php echo Song::play_url($song->id); ?>" title="<?php echo scrub_out($song->title); ?>"><?php echo $song->f_title; ?></a></td> +<td class="cel_song"><?php echo $song->f_link; ?></a></td> <td class="cel_artist"><?php echo $song->f_artist_link; ?></td> <td class="cel_album"><?php echo $song->f_album_link; ?></td> <td class="cel_tags"><?php echo $song->f_tags; ?></td> |