Skip to content

Commit c8967c2

Browse files
committed
quick fix for player favorite
1 parent 419aa15 commit c8967c2

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

app/index.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,7 @@ <h4 id="playerUser" class="player_user" ng-click="goToUser($event)"></h4>
150150
<span class="player_duration" id="player-duration"></span>
151151
</div>
152152
<div class="player_actions">
153-
<span class="player_favorite" ng-click="favorite($event)" ng-if="isSongPlaying">
153+
<span class="player_favorite" ng-click="favorite($event)" ng-hide="!isSongPlaying">
154154
&nbsp;<i class="fa fa-heart"></i>
155155
</span>
156156
</div>

app/public/js/common/playerService.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -161,6 +161,10 @@ app.factory('playerService', function(
161161

162162
$rootScope.isSongPlaying = true;
163163
$rootScope.$broadcast('activateQueue');
164+
165+
// remove the active class from player favorite icon before play new song
166+
// TODO: this should check if the current song is already favorited
167+
document.querySelector('.player_favorite').classList.remove('active');
164168
};
165169

166170
/**

0 commit comments

Comments
 (0)