Canvas: Button API Editor visual feedback on response (#76499)

public/app/features/canvas/element.ts

@@ -92,4 +92,4 @@ export interface CanvasElementItem<TConfig = any, TData = any> extends RegistryI
 export const defaultBgColor = '#D9D9D9';
 export const defaultTextColor = '#000000';
 export const defaultLightTextColor = '#F0F4FD';
-export const defaultThemeTextColor = config.theme2.colors.background.primary;
+export const defaultThemeTextColor = config.theme2.colors.text.primary;

public/app/features/canvas/elements/button.tsx

@@ -1,11 +1,10 @@
 import { css } from '@emotion/css';
-import React, { PureComponent } from 'react';
+import React from 'react';
 
 import { GrafanaTheme2 } from '@grafana/data';
 import { PluginState } from '@grafana/data/src';
 import { TextDimensionMode } from '@grafana/schema';
-import { Button, stylesFactory } from '@grafana/ui';
-import { config } from 'app/core/config';
+import { Button, Spinner, useStyles2 } from '@grafana/ui';
 import { DimensionContext } from 'app/features/dimensions/context';
 import { ColorDimensionEditor } from 'app/features/dimensions/editors';
 import { TextDimensionEditor } from 'app/features/dimensions/editors/TextDimensionEditor';
@@ -40,26 +39,39 @@ export const defaultStyleConfig: ButtonStyleConfig = {
   variant: 'primary',
 };
 
-class ButtonDisplay extends PureComponent<CanvasElementProps<ButtonConfig, ButtonData>> {
-  render() {
-    const { data } = this.props;
-    const styles = getStyles(config.theme2, data);
+const ButtonDisplay = ({ data }: CanvasElementProps<ButtonConfig, ButtonData>) => {
+  const styles = useStyles2(getStyles, data);
 
-    const onClick = () => {
-      if (data?.api && data?.api?.endpoint) {
-        callApi(data.api);
-      }
-    };
+  const [isLoading, setIsLoading] = React.useState(false);
 
-    return (
-      <Button type="submit" variant={data?.style?.variant} onClick={onClick} className={styles.button}>
+  const updateLoadingStateCallback = (loading: boolean) => {
+    setIsLoading(loading);
+  };
+
+  const onClick = () => {
+    if (data?.api && data?.api?.endpoint) {
+      setIsLoading(true);
+      callApi(data.api, updateLoadingStateCallback);
+    }
+  };
+
+  return (
+    <Button
+      type="submit"
+      variant={data?.style?.variant}
+      onClick={onClick}
+      className={styles.button}
+      disabled={!data?.api?.endpoint}
+    >
+      <span>
+        {isLoading && <Spinner inline={true} className={styles.buttonSpinner} />}
         {data?.text}
-      </Button>
-    );
-  }
-}
+      </span>
+    </Button>
+  );
+};
 
-const getStyles = stylesFactory((theme: GrafanaTheme2, data: ButtonData | undefined) => ({
+const getStyles = (theme: GrafanaTheme2, data: ButtonData | undefined) => ({
   button: css({
     height: '100%',
     width: '100%',
@@ -67,12 +79,16 @@ const getStyles = stylesFactory((theme: GrafanaTheme2, data: ButtonData | undefi
 
     '> span': {
       display: 'inline-grid',
+      gridAutoFlow: 'column',
       textAlign: data?.align,
       fontSize: `${data?.size}px`,
       color: data?.color,
     },
   }),
-}));
+  buttonSpinner: css({
+    marginRight: theme.spacing(0.5),
+  }),
+});
 
 export const buttonItem: CanvasElementItem<ButtonConfig, ButtonData> = {
   id: 'button',
@@ -137,7 +153,7 @@ export const buttonItem: CanvasElementItem<ButtonConfig, ButtonData> = {
     const data: ButtonData = {
       text: cfg?.text ? ctx.getText(cfg.text).value() : '',
       align: cfg.align ?? Align.Center,
-      size: cfg.size,
+      size: cfg.size ?? 14,
       api: getCfgApi(),
       style: cfg?.style ?? defaultStyleConfig,
     };

public/app/plugins/panel/canvas/editor/element/APIEditor.tsx

@@ -132,7 +132,7 @@ export function APIEditor({ value, context, onChange }: Props) {
   const renderTestAPIButton = (api: APIEditorConfig) => {
     if (api && api.endpoint) {
       return (
-        <Button onClick={() => callApi(api, true)} title="Test API">
+        <Button onClick={() => callApi(api)} title="Test API">
           Test API
         </Button>
       );
@@ -158,36 +158,36 @@ export function APIEditor({ value, context, onChange }: Props) {
           <RadioButtonGroup value={value?.method} options={httpMethodOptions} onChange={onMethodChange} fullWidth />
         </InlineField>
       </InlineFieldRow>
+      {value?.method === HttpRequestMethod.POST && (
+        <InlineFieldRow>
+          <InlineField label="Content-Type" labelWidth={LABEL_WIDTH} grow={true}>
+            <Select
+              options={contentTypeOptions}
+              allowCustomValue={true}
+              formatCreateLabel={formatCreateLabel}
+              value={value?.contentType}
+              onChange={onContentTypeChange}
+            />
+          </InlineField>
+        </InlineFieldRow>
+      )}
+
+      <br />
       <Field label="Query parameters">
         <ParamsEditor value={value?.queryParams ?? []} onChange={onQueryParamsChange} />
       </Field>
       <Field label="Header parameters">
         <ParamsEditor value={value?.headerParams ?? []} onChange={onHeaderParamsChange} />
       </Field>
-      {value?.method === HttpRequestMethod.POST && (
-        <>
-          <InlineFieldRow>
-            <InlineField label="Content-Type" labelWidth={LABEL_WIDTH} grow={true}>
-              <Select
-                options={contentTypeOptions}
-                allowCustomValue={true}
-                formatCreateLabel={formatCreateLabel}
-                value={value?.contentType}
-                onChange={onContentTypeChange}
-              />
-            </InlineField>
-          </InlineFieldRow>
-          {value?.contentType && (
-            <Field label="Payload">
-              <StringValueEditor
-                context={context}
-                value={value?.data ?? '{}'}
-                onChange={onDataChange}
-                item={{ ...dummyStringSettings, settings: { useTextarea: true } }}
-              />
-            </Field>
-          )}
-        </>
+      {value?.method === HttpRequestMethod.POST && value?.contentType && (
+        <Field label="Payload">
+          <StringValueEditor
+            context={context}
+            value={value?.data ?? '{}'}
+            onChange={onDataChange}
+            item={{ ...dummyStringSettings, settings: { useTextarea: true } }}
+          />
+        </Field>
       )}
       {renderTestAPIButton(value)}
       <br />

public/app/plugins/panel/canvas/editor/element/utils.ts

@@ -7,11 +7,14 @@ import { HttpRequestMethod } from '../../panelcfg.gen';
 
 import { APIEditorConfig } from './APIEditor';
 
-export const callApi = (api: APIEditorConfig, isTest = false) => {
+type IsLoadingCallback = (loading: boolean) => void;
+
+export const callApi = (api: APIEditorConfig, updateLoadingStateCallback?: IsLoadingCallback) => {
   if (api && api.endpoint) {
     // If API endpoint origin matches Grafana origin, don't call it.
     if (requestMatchesGrafanaOrigin(api.endpoint)) {
       appEvents.emit(AppEvents.alertError, ['Cannot call API at Grafana origin.']);
+      updateLoadingStateCallback && updateLoadingStateCallback(false);
       return;
     }
     const request = getRequest(api);
@@ -20,15 +23,12 @@ export const callApi = (api: APIEditorConfig, isTest = false) => {
       .fetch(request)
       .subscribe({
         error: (error) => {
-          if (isTest) {
-            appEvents.emit(AppEvents.alertError, ['Error has occurred: ', JSON.stringify(error)]);
-            console.error(error);
-          }
+          appEvents.emit(AppEvents.alertError, ['An error has occurred: ', JSON.stringify(error)]);
+          updateLoadingStateCallback && updateLoadingStateCallback(false);
         },
         complete: () => {
-          if (isTest) {
-            appEvents.emit(AppEvents.alertSuccess, ['Test successful']);
-          }
+          appEvents.emit(AppEvents.alertSuccess, ['API call was successful']);
+          updateLoadingStateCallback && updateLoadingStateCallback(false);
         },
       });
   }