KEYCLOAK-1434 Change edit action to buttons to make sure styles are correct

forms/common-themes/src/main/resources/theme/base/admin/resources/js/app.js

@@ -1843,3 +1843,19 @@ module.directive('kcTooltip', function($compile) {
             }
         };
 });
+
+module.directive( 'kcOpen', function ( $location ) {
+    return function ( scope, element, attrs ) {
+        var path;
+
+        attrs.$observe( 'kcOpen', function (val) {
+            path = val;
+        });
+
+        element.bind( 'click', function () {
+            scope.$apply( function () {
+                $location.path(path);
+            });
+        });
+    };
+});

forms/common-themes/src/main/resources/theme/base/admin/resources/partials/client-list.html

@@ -41,7 +41,7 @@
                 <span data-ng-hide="client.baseUrl">Not defined</span>
             </td>
             <td class="kc-action-cell">
-                <a class="btn btn-default btn-block btn-sm" href="#/realms/{{realm.realm}}/clients/{{client.id}}">Edit</a>
+                <button class="btn btn-default btn-block btn-sm" kc-open="/realms/{{realm.realm}}/clients/{{client.id}}">Edit</button>
             </td>
         </tr>
         <tr data-ng-show="(clients | filter:search).length == 0">

forms/common-themes/src/main/resources/theme/base/admin/resources/partials/client-role-list.html

@@ -10,7 +10,7 @@
     <table class="table table-striped table-bordered">
         <thead>
         <tr>
-            <th class="kc-table-actions" colspan="3" data-ng-show="access.manageClients">
+            <th class="kc-table-actions" colspan="4" data-ng-show="access.manageClients">
                 <div class="pull-right">
                     <a class="btn btn-default" href="#/create/role/{{realm.realm}}/clients/{{client.id}}">Add Role</a>
                 </div>
@@ -20,6 +20,7 @@
             <th>Role Name</th>
             <th>Composite</th>
             <th>Description</th>
+            <th>Actions</th>
         </tr>
         </thead>
         <tbody>
@@ -27,6 +28,9 @@
             <td><a href="#/realms/{{realm.realm}}/clients/{{client.id}}/roles/{{role.id}}">{{role.name}}</a></td>
             <td>{{role.composite}}</td>
             <td>{{role.description}}</td>
+            <td class="kc-action-cell">
+                <button class="btn btn-default btn-block btn-sm" kc-open="/realms/{{realm.realm}}/clients/{{client.id}}/roles/{{role.id}}">Edit</button>
+            </td>
         </tr>
         <tr data-ng-show="!roles || roles.length == 0">
             <td>No client roles available</td>

forms/common-themes/src/main/resources/theme/base/admin/resources/partials/realm-identity-provider.html

@@ -35,7 +35,7 @@
                         <td>{{identityProvider.enabled}}</td>
                         <td>{{identityProvider.config.guiOrder}}</td>
                         <td class="kc-action-cell">
-                            <a class="btn btn-default btn-block btn-sm" href="#/realms/{{realm.realm}}/identity-provider-settings/provider/{{identityProvider.providerId}}/{{identityProvider.alias}}">Edit</a>
+                            <button class="btn btn-default btn-block btn-sm" kc-open="/realms/{{realm.realm}}/identity-provider-settings/provider/{{identityProvider.providerId}}/{{identityProvider.alias}}">Edit</button>
                         </td>
                     </tr>
                     </tbody>

forms/common-themes/src/main/resources/theme/base/admin/resources/partials/role-list.html

@@ -39,7 +39,7 @@
             <td>{{role.composite}}</td>
             <td>{{role.description}}</td>
             <td class="kc-action-cell">
-                <a class="btn btn-default btn-block btn-sm" href="#/realms/{{realm.realm}}/roles/{{role.id}}">Edit</a>
+                <button class="btn btn-default btn-block btn-sm" kc-open="/realms/{{realm.realm}}/roles/{{role.id}}">Edit</button>
             </td>
         </tr>
         <tr data-ng-show="(roles | filter:{name: searchQuery}).length == 0">

forms/common-themes/src/main/resources/theme/base/admin/resources/partials/user-federation.html

@@ -31,7 +31,7 @@
             <td>{{instance.providerName|capitalize}}</td>
             <td>{{instance.priority}}</td>
             <td class="kc-action-cell">
-                <a class="btn btn-default btn-block btn-sm" href="#/realms/{{realm.realm}}/user-federation/providers/{{instance.providerName}}/{{instance.id}}">Edit</a>
+                <button class="btn btn-default btn-block btn-sm" kc-open="/realms/{{realm.realm}}/user-federation/providers/{{instance.providerName}}/{{instance.id}}">Edit</button>
             </td>
          </tr>
         <tr data-ng-show="!instances || instances.length == 0">

forms/common-themes/src/main/resources/theme/base/admin/resources/partials/user-list.html

@@ -51,7 +51,7 @@
             <td>{{user.firstName}}</td>
             <td>{{user.email}}</td>
             <td class="kc-action-cell">
-                <a class="btn btn-default btn-block btn-sm" href="#/realms/{{realm.realm}}/clients/{{client.id}}">Edit</a>
+                <button class="btn btn-default btn-block btn-sm" kc-open="/realms/{{realm.realm}}/users/{{user.id}}">Edit</button>
             </td>
             <td data-ng-show="access.impersonation" class="kc-action-cell">
                 <button class="btn btn-default btn-block btn-sm" data-ng-click="impersonate(user.id)" tooltip="Login as this user.  If user is in same realm as you, your current login session will be logged out before you are logged in as this user.">Impersonate</button>

forms/common-themes/src/main/resources/theme/keycloak/admin/resources/css/styles.css

@@ -313,7 +313,3 @@ h1 i {
     right: 0;
     bottom: 0;
 }
-
-.kc-action-cell a {
-    padding-top: 4px;
-}