vbatts/quay
Archived
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #1758 from coreos-inc/hide-star

Browse source 
Hide repo star control when not logged in
This commit is contained in:
josephschorr 2016-08-23 13:16:42 -04:00 committed by GitHub
commit c4d54ceba7
4 changed files with 15 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
static/directives/repo-list-table.html
View file

@ -24,7 +24,8 @@
</td> </td>
<td class="hidden-xs" <td class="hidden-xs"
ng-class="tablePredicateClass('is_starred', options.predicate, options.reverse)" ng-class="tablePredicateClass('is_starred', options.predicate, options.reverse)"
style="width: 70px"> style="width: 70px"
ng-if="loggedIn">
<a ng-click="orderBy('is_starred')">Star</a> <a ng-click="orderBy('is_starred')">Star</a>
</td> </td>
</thead> </thead>
@ -48,9 +49,10 @@
<span class="strength-indicator" value="repository.popularity" maximum="maxPopularity" <span class="strength-indicator" value="repository.popularity" maximum="maxPopularity"
log-base="10"></span> log-base="10"></span>
</td> </td>
<td> <td ng-show="loggedIn">
<span class="repo-star" repository="repository" <span class="repo-star" repository="repository"
star-toggled="starToggled({'repository': repository})"></span></td> star-toggled="starToggled({'repository': repository})"></span>
</td>
</tr> </tr>
</tbody> </tbody>
</table> </table>

2
static/directives/repo-star.html
View file

@ -1,5 +1,5 @@
<span class="repo-star-element"> <span class="repo-star-element">
<i ng-class="repository.is_starred ? 'starred fa fa-star' : 'fa fa-star-o'" <i ng-class="repository.is_starred ? 'starred fa fa-star' : 'fa fa-star-o'"
class="star-icon" ng-click="toggleStar()"> class="star-icon" ng-click="toggleStar()" ng-show="loggedIn">
</i> </i>
</span> </span>

4
static/js/directives/ui/repo-list-table.js
View file

@ -13,7 +13,7 @@ angular.module('quay').directive('repoListTable', function () {
'namespaces': '=namespaces', 'namespaces': '=namespaces',
'starToggled': '&starToggled' 'starToggled': '&starToggled'
}, },
controller: function($scope, $element, $filter, TableService) { controller: function($scope, $element, $filter, TableService, UserService) {
$scope.repositories = null; $scope.repositories = null;
$scope.orderedRepositories = []; $scope.orderedRepositories = [];
@ -93,6 +93,8 @@ angular.module('quay').directive('repoListTable', function () {
}); });
buildOrderedRepositories(); buildOrderedRepositories();
$scope.loggedIn = !UserService.currentUser().anonymous;
}, /* deep */ true); }, /* deep */ true);
} }
}; };

6
static/js/directives/ui/repo-star.js
View file

@ -13,6 +13,8 @@ angular.module('quay').directive('repoStar', function () {
starToggled: '&starToggled' starToggled: '&starToggled'
}, },
controller: function($scope, $element, UserService, ApiService) { controller: function($scope, $element, UserService, ApiService) {
$scope.loggedIn = false;
// Star a repository or unstar a repository. // Star a repository or unstar a repository.
$scope.toggleStar = function() { $scope.toggleStar = function() {
if ($scope.repository.is_starred) { if ($scope.repository.is_starred) {
@ -46,6 +48,10 @@ angular.module('quay').directive('repoStar', function () {
$scope.starToggled({'repository': $scope.repository}); $scope.starToggled({'repository': $scope.repository});
}, ApiService.errorDisplay('Could not unstar repository')); }, ApiService.errorDisplay('Could not unstar repository'));
}; };
$scope.$watch('repository', function() {
$scope.loggedIn = !UserService.currentUser().anonymous;
});
} }
}; };