From 2a02b38888743450a4c7dd9c8411c7ee12df03d0 Mon Sep 17 00:00:00 2001
From: Erik Jan de Wit <erikjan.dewit@gmail.com>
Date: Fri, 23 Apr 2021 11:55:15 +0200
Subject: [PATCH 1/3] fixing PR comment:

https://github.com/keycloak/keycloak-admin-ui/pull/474#issuecomment-825417974
---
 src/clients/add/CapabilityConfig.tsx | 9 +++++----
 src/clients/messages.json            | 4 ++--
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/src/clients/add/CapabilityConfig.tsx b/src/clients/add/CapabilityConfig.tsx
index 7f1f5521d7..6bdb510cc5 100644
--- a/src/clients/add/CapabilityConfig.tsx
+++ b/src/clients/add/CapabilityConfig.tsx
@@ -187,11 +187,12 @@ export const CapabilityConfig = ({
                           label={t("serviceAccount")}
                           id="kc-flow-service-account"
                           name="serviceAccountsEnabled"
-                          isChecked={value}
-                          onChange={onChange}
-                          isDisabled={
-                            !clientAuthentication || clientAuthorization
+                          isChecked={
+                            value ||
+                            (clientAuthentication && clientAuthorization)
                           }
+                          onChange={onChange}
+                          isDisabled={clientAuthorization}
                         />
                         <HelpItem
                           helpText="clients-help:serviceAccount"
diff --git a/src/clients/messages.json b/src/clients/messages.json
index 457a03d864..2089ef937b 100644
--- a/src/clients/messages.json
+++ b/src/clients/messages.json
@@ -103,8 +103,8 @@
     "authentication": "Authentication",
     "authenticationFlow": "Authentication flow",
     "standardFlow": "Standard flow",
-    "directAccess": "Direct access",
-    "serviceAccount": "Service account roles",
+    "directAccess": "Direct access grants",
+    "serviceAccount": "Service account",
     "enableServiceAccount": "Enable service account roles",
     "assignRolesTo": "Assign roles to {{client}} account",
     "searchByRoleName": "Search by role name",

From 1521024abfd861c89fdbbcf3173aaa9ad04e5cdb Mon Sep 17 00:00:00 2001
From: Erik Jan de Wit <erikjan.dewit@gmail.com>
Date: Mon, 26 Apr 2021 09:40:59 +0200
Subject: [PATCH 2/3] fixed the wrong comment

---
 src/clients/add/CapabilityConfig.tsx | 14 ++++++++------
 src/clients/add/NewClientForm.tsx    |  1 -
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/src/clients/add/CapabilityConfig.tsx b/src/clients/add/CapabilityConfig.tsx
index 6bdb510cc5..8f2b58f339 100644
--- a/src/clients/add/CapabilityConfig.tsx
+++ b/src/clients/add/CapabilityConfig.tsx
@@ -26,7 +26,7 @@ export const CapabilityConfig = ({
   const { control, watch, setValue } = useFormContext<ClientForm>();
   const protocol = type || watch("protocol");
   const clientAuthentication = watch("publicClient");
-  const clientAuthorization = watch("authorizationServicesEnabled");
+  const authorization = watch("authorizationServicesEnabled");
 
   return (
     <FormAccess isHorizontal role="manage-clients" unWrap={unWrap}>
@@ -47,7 +47,7 @@ export const CapabilityConfig = ({
             >
               <Controller
                 name="publicClient"
-                defaultValue={true}
+                defaultValue={false}
                 control={control}
                 render={({ onChange, value }) => (
                   <Switch
@@ -85,7 +85,7 @@ export const CapabilityConfig = ({
                     name="authorizationServicesEnabled"
                     label={t("common:on")}
                     labelOff={t("common:off")}
-                    isChecked={value}
+                    isChecked={value && !clientAuthentication}
                     onChange={(value) => {
                       onChange(value);
                       if (value) {
@@ -188,11 +188,13 @@ export const CapabilityConfig = ({
                           id="kc-flow-service-account"
                           name="serviceAccountsEnabled"
                           isChecked={
-                            value ||
-                            (clientAuthentication && clientAuthorization)
+                            value || (clientAuthentication && authorization)
                           }
                           onChange={onChange}
-                          isDisabled={clientAuthorization}
+                          isDisabled={
+                            (clientAuthentication && !authorization) ||
+                            (!clientAuthentication && authorization)
+                          }
                         />
                         <HelpItem
                           helpText="clients-help:serviceAccount"
diff --git a/src/clients/add/NewClientForm.tsx b/src/clients/add/NewClientForm.tsx
index d840706d75..76a0e875dc 100644
--- a/src/clients/add/NewClientForm.tsx
+++ b/src/clients/add/NewClientForm.tsx
@@ -7,7 +7,6 @@ import {
   WizardFooter,
   WizardContextConsumer,
   Button,
-  WizardStepFunctionType,
 } from "@patternfly/react-core";
 import { useTranslation } from "react-i18next";
 import { FormProvider, useForm } from "react-hook-form";

From 27c25472eea36fad56aeb2e3c5a0ade39a154d2f Mon Sep 17 00:00:00 2001
From: Erik Jan de Wit <erikjan.dewit@gmail.com>
Date: Mon, 26 Apr 2021 13:48:09 +0200
Subject: [PATCH 3/3] small fixes

---
 src/clients/add/CapabilityConfig.tsx  | 28 ++++++++++++++++++++-------
 src/clients/add/capability-config.css |  5 +++++
 src/clients/messages.json             |  2 +-
 3 files changed, 27 insertions(+), 8 deletions(-)
 create mode 100644 src/clients/add/capability-config.css

diff --git a/src/clients/add/CapabilityConfig.tsx b/src/clients/add/CapabilityConfig.tsx
index 8f2b58f339..3ba9036af5 100644
--- a/src/clients/add/CapabilityConfig.tsx
+++ b/src/clients/add/CapabilityConfig.tsx
@@ -1,5 +1,6 @@
 import React from "react";
 import { useTranslation } from "react-i18next";
+import { Controller, useFormContext } from "react-hook-form";
 import {
   FormGroup,
   Switch,
@@ -8,11 +9,13 @@ import {
   GridItem,
   InputGroup,
 } from "@patternfly/react-core";
-import { Controller, useFormContext } from "react-hook-form";
+
 import { FormAccess } from "../../components/form-access/FormAccess";
 import { ClientForm } from "../ClientDetails";
 import { HelpItem } from "../../components/help-enabler/HelpItem";
 
+import "./capability-config.css";
+
 type CapabilityConfigProps = {
   unWrap?: boolean;
   protocol?: string;
@@ -29,7 +32,12 @@ export const CapabilityConfig = ({
   const authorization = watch("authorizationServicesEnabled");
 
   return (
-    <FormAccess isHorizontal role="manage-clients" unWrap={unWrap}>
+    <FormAccess
+      isHorizontal
+      role="manage-clients"
+      unWrap={unWrap}
+      className="keycloak__capability-config__form"
+    >
       <>
         {protocol === "openid-connect" && (
           <>
@@ -57,7 +65,13 @@ export const CapabilityConfig = ({
                     label={t("common:on")}
                     labelOff={t("common:off")}
                     isChecked={!value}
-                    onChange={(value) => onChange(!value)}
+                    onChange={(value) => {
+                      onChange(!value);
+                      if (!value) {
+                        setValue("authorizationServicesEnabled", false);
+                        setValue("serviceAccountsEnabled", false);
+                      }
+                    }}
                   />
                 )}
               />
@@ -103,7 +117,7 @@ export const CapabilityConfig = ({
               fieldId="kc-flow"
             >
               <Grid>
-                <GridItem lg={3} sm={6}>
+                <GridItem lg={4} sm={6}>
                   <Controller
                     name="standardFlowEnabled"
                     defaultValue={true}
@@ -127,7 +141,7 @@ export const CapabilityConfig = ({
                     )}
                   />
                 </GridItem>
-                <GridItem lg={9} sm={6}>
+                <GridItem lg={8} sm={6}>
                   <Controller
                     name="directAccessGrantsEnabled"
                     defaultValue={true}
@@ -151,7 +165,7 @@ export const CapabilityConfig = ({
                     )}
                   />
                 </GridItem>
-                <GridItem lg={3} sm={6}>
+                <GridItem lg={4} sm={6}>
                   <Controller
                     name="implicitFlowEnabled"
                     defaultValue={false}
@@ -175,7 +189,7 @@ export const CapabilityConfig = ({
                     )}
                   />
                 </GridItem>
-                <GridItem lg={9} sm={6}>
+                <GridItem lg={8} sm={6}>
                   <Controller
                     name="serviceAccountsEnabled"
                     defaultValue={false}
diff --git a/src/clients/add/capability-config.css b/src/clients/add/capability-config.css
new file mode 100644
index 0000000000..46545f21c1
--- /dev/null
+++ b/src/clients/add/capability-config.css
@@ -0,0 +1,5 @@
+@media (min-width: 768px) {
+  .keycloak__capability-config__form .pf-c-form__group {
+    --pf-c-form--m-horizontal__group-label--md--GridColumnWidth: 10rem;
+  }
+}
diff --git a/src/clients/messages.json b/src/clients/messages.json
index 2089ef937b..65b238e84e 100644
--- a/src/clients/messages.json
+++ b/src/clients/messages.json
@@ -104,7 +104,7 @@
     "authenticationFlow": "Authentication flow",
     "standardFlow": "Standard flow",
     "directAccess": "Direct access grants",
-    "serviceAccount": "Service account",
+    "serviceAccount": "Service accounts",
     "enableServiceAccount": "Enable service account roles",
     "assignRolesTo": "Assign roles to {{client}} account",
     "searchByRoleName": "Search by role name",