Merge pull request #961 from taigaio/us/273/ui-changes

Update UI for plugins and project transfer

app/modules/navigation-bar/dropdown-user/dropdown-user.jade

@@ -30,8 +30,6 @@ div.navbar-dropdown.dropdown-user
                 ng-class="{active: plugin.slug == currentPlugin.slug}"
             )
                 span.title {{ plugin.name }}
-                span.new(translate="PROJECT.NAVIGATION.NEW_ITEM")
-
         li
             a(
                 href="#",

app/modules/projects/transfer/transfer-project.controller.coffee

@@ -29,10 +29,9 @@ class TransferProject
         "$tgNavUrls",
         "$translate",
         "$tgConfirm"
-        "$tgLoading"
     ]
 
-    constructor: (@routeParams, @projectService, @location, @authService, @currentUserService, @navUrls, @translate, @confirmService, @loading) ->
+    constructor: (@routeParams, @projectService, @location, @authService, @currentUserService, @navUrls, @translate, @confirmService) ->
 
     initialize: () ->
         @.projectId = @.project.get("id")
@@ -73,31 +72,25 @@ class TransferProject
 
         @.validNumberOfMemberships = maxMemberships == null || @.project.get('total_memberships') <= maxMemberships
 
-    transferAccept: (token, reason, $event) ->
-        target = angular.element($event.currentTarget)
-        currentLoading = @loading()
-            .target(target)
-            .start()
+    transferAccept: (token, reason) ->
+        @.loadingAccept = true
         return @projectService.transferAccept(@.project.get("id"), token, reason).then () =>
             newUrl = @navUrls.resolve("project-admin-project-profile-details", {
                 project: @.project.get("slug")
             })
-            currentLoading.finish()
+            @.loadingAccept = false
             @location.path(newUrl)
 
             @confirmService.notify("success", @translate.instant("ADMIN.PROJECT_TRANSFER.ACCEPTED_PROJECT_OWNERNSHIP"), '', 5000)
             return
 
-    transferReject: (token, reason, $event) ->
-        target = $($event.target)
-        currentLoading = @loading()
-            .target(target)
-            .start()
+    transferReject: (token, reason) ->
+        @.loadingReject = true
         return @projectService.transferReject(@.project.get("id"), token, reason).then () =>
             newUrl = @navUrls.resolve("home", {
                 project: @project.get("slug")
             })
-            currentLoading.finish()
+            @.loadingReject = false
             @location.path(newUrl)
             @confirmService.notify("success", @translate.instant("ADMIN.PROJECT_TRANSFER.REJECTED_PROJECT_OWNERNSHIP"), '', 5000)
 

app/modules/projects/transfer/transfer-project.jade

@@ -57,6 +57,7 @@
                 href="#"
                 title="{{'ADMIN.PROJECT_TRANSFER.REJECT' | translate}}"
                 translate="ADMIN.PROJECT_TRANSFER.REJECT"
+                tg-loading="vm.loadingReject"
             )
 
             a.button.button-green(
@@ -64,6 +65,7 @@
                 href="#"
                 title="{{'ADMIN.PROJECT_TRANSFER.ACCEPT' | translate}}"
                 translate="ADMIN.PROJECT_TRANSFER.ACCEPT"
+                tg-loading="vm.loadingAccept"
             )
 
     tg-cant-own-project-explanation(ng-if="!vm.canBeOwnedByUser.valid")