changed to be progressive added help to type

and made `openid-connect` the default

fixes: #446

src/clients/add/GeneralSettings.tsx

@@ -11,9 +11,10 @@ import { Controller, useFormContext } from "react-hook-form";
 import { useLoginProviders } from "../../context/server-info/ServerInfoProvider";
 import { ClientDescription } from "../ClientDescription";
 import { FormAccess } from "../../components/form-access/FormAccess";
+import { HelpItem } from "../../components/help-enabler/HelpItem";
 
 export const GeneralSettings = () => {
-  const { t } = useTranslation();
+  const { t } = useTranslation("clients");
   const { errors, control } = useFormContext();
 
   const providers = useLoginProviders();
@@ -22,30 +23,32 @@ export const GeneralSettings = () => {
   return (
     <FormAccess isHorizontal role="manage-clients">
       <FormGroup
-        label="Client Type"
+        label={t("clientType")}
         fieldId="kc-type"
-        isRequired
-        helperTextInvalid={t("common:required")}
         validated={errors.protocol ? "error" : "default"}
+        labelIcon={
+          <HelpItem
+            helpText="clients-help:clientType"
+            forLabel={t("clientType")}
+            forID="kc-type"
+          />
+        }
       >
         <Controller
           name="protocol"
           defaultValue=""
           control={control}
-          rules={{ required: true }}
           render={({ onChange, value }) => (
             <Select
               id="kc-type"
-              required
               onToggle={() => isOpen(!open)}
-              onSelect={(_, value, isPlaceholder) => {
+              onSelect={(_, value) => {
-                onChange(isPlaceholder ? "" : (value as string));
+                onChange(value as string);
                 isOpen(false);
               }}
               selections={value}
               variant={SelectVariant.single}
               aria-label={t("selectEncryptionType")}
-              placeholderText={t("common:selectOne")}
               isOpen={open}
             >
               {providers.map((option) => (

src/clients/add/NewClientForm.tsx

@@ -25,8 +25,9 @@ export const NewClientForm = () => {
   const adminClient = useAdminClient();
   const history = useHistory();
 
+  const [showCapabilityConfig, setShowCapabilityConfig] = useState(false);
   const [client, setClient] = useState<ClientRepresentation>({
-    protocol: "",
+    protocol: "openid-connect",
     clientId: "",
     name: "",
     description: "",
@@ -62,6 +63,7 @@ export const NewClientForm = () => {
                 onClick={async () => {
                   if (await methods.trigger()) {
                     setClient({ ...client, ...methods.getValues() });
+                    setShowCapabilityConfig(true);
                     onNext();
                   }
                 }}
@@ -106,13 +108,21 @@ export const NewClientForm = () => {
             mainAriaLabel={`${title} content`}
             steps={[
               {
+                id: "generalSettings",
                 name: t("generalSettings"),
                 component: <GeneralSettings />,
               },
+              ...(showCapabilityConfig
+                ? [
                     {
+                      id: "capabilityConfig",
                       name: t("capabilityConfig"),
-                component: <CapabilityConfig protocol={client.protocol} />,
+                      component: (
+                        <CapabilityConfig protocol={client.protocol} />
+                      ),
                     },
+                  ]
+                : []),
             ]}
             footer={<Footer />}
             onSave={save}

src/clients/help.json

@@ -1,5 +1,6 @@
 {
   "clients-help": {
+    "clientType": "'OpenID connect' allows Clients to verify the identity of the End-User based on the authentication performed by an Authorization Server.'SAML' enables web-based authentication and authorization scenarios including cross-domain single sign-on (SSO) and uses security tokens containing assertions to pass information.",
     "serviceAccount": "Allows you to authenticate this client to Keycloak and retrieve access token dedicated to this client. In terms of OAuth2 specification, this enables support of 'Client Credentials Grant' for this client.",
     "directAccess": "This enables support for Direct Access Grants, which means that client has access to username/password of user and exchange it directly with Keycloak server for access token. In terms of OAuth2 specification, this enables support of 'Resource Owner Password Credentials Grant' for this client.",
     "standardFlow": "This enables standard OpenID Connect redirect based authentication with authorization code. In terms of OpenID Connect or OAuth2 specifications, this enables support of 'Authorization Code Flow' for this client.",

src/clients/messages.json

@@ -1,5 +1,6 @@
 {
   "clients": {
+    "clientType": "Client type",
     "clientAuthorization": "Client authorization",
     "implicitFlow": "Implicit flow",
     "createClient": "Create client",