Skip to content

Commit 613b818

Browse files
fit and finish enhancements (#428) (#430)
(cherry picked from commit 3bf513a) Signed-off-by: saimedhi <saimedhi@amazon.com> Signed-off-by: Sai Medhini Reddy Maryada <117196660+saimedhi@users.noreply.github.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
1 parent 2aab2ff commit 613b818

26 files changed

+90
-81
lines changed

public/general_components/multi_select_filter.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ export function MultiSelectFilter(props: MultiSelectFilterProps) {
7676
}
7777
isOpen={isPopoverOpen}
7878
closePopover={onPopoverClose}
79-
panelPaddingSize="none"
79+
panelPaddingSize="s"
8080
>
8181
<div className="euiFilterSelect__items multi-select-filter--width">
8282
{filters.map((filter, index) => (

public/general_components/processors_title.tsx

+7-7
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
*/
55

66
import React from 'react';
7-
import { EuiFlexItem, EuiTitle } from '@elastic/eui';
7+
import { EuiFlexItem, EuiText } from '@elastic/eui';
88

99
interface ProcessorsTitleProps {
1010
title: string;
@@ -17,17 +17,17 @@ interface ProcessorsTitleProps {
1717
export function ProcessorsTitle(props: ProcessorsTitleProps) {
1818
return (
1919
<EuiFlexItem grow={false}>
20-
<EuiTitle size="s">
20+
<EuiText size="s">
2121
<div>
22-
<h2
22+
<h3
2323
style={{ display: 'inline-block' }}
24-
>{`${props.title} (${props.processorCount}) -`}</h2>
24+
>{`${props.title} (${props.processorCount}) -`}</h3>
2525
&nbsp;
26-
<h2 style={{ display: 'inline-block', fontStyle: 'italic' }}>
26+
<h3 style={{ display: 'inline-block', fontStyle: 'italic' }}>
2727
optional
28-
</h2>
28+
</h3>
2929
</div>
30-
</EuiTitle>
30+
</EuiText>
3131
</EuiFlexItem>
3232
);
3333
}

public/general_components/service_card/search_studio_card.tsx

+8-4
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,12 @@
44
*/
55

66
import React from 'react';
7-
import { EuiButton, EuiFlexGroup, EuiFlexItem, EuiIcon } from '@elastic/eui';
7+
import {
8+
EuiSmallButton,
9+
EuiFlexGroup,
10+
EuiFlexItem,
11+
EuiIcon,
12+
} from '@elastic/eui';
813
import { i18n } from '@osd/i18n';
914
import { PLUGIN_ID } from '../../../common';
1015
import { ContentManagementPluginStart } from '../../../../../src/plugins/content_management/public';
@@ -26,16 +31,15 @@ export const registerSearchStudioCard = (
2631
const footer = (
2732
<EuiFlexGroup justifyContent="flexEnd">
2833
<EuiFlexItem grow={false}>
29-
<EuiButton
30-
size="s"
34+
<EuiSmallButton
3135
onClick={() => {
3236
core.application.navigateToApp(PLUGIN_ID);
3337
}}
3438
>
3539
{i18n.translate('flowFrameworkDashboards.searchStudioCard.footer', {
3640
defaultMessage: 'Try OpenSearch Studio',
3741
})}
38-
</EuiButton>
42+
</EuiSmallButton>
3943
</EuiFlexItem>
4044
</EuiFlexGroup>
4145
);

public/pages/workflow_detail/components/export_modal.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ export function ExportModal(props: ExportModalProps) {
8383
<EuiModalBody>
8484
<EuiFlexGroup direction="column">
8585
<EuiFlexItem grow={false}>
86-
<EuiText>
86+
<EuiText size="s">
8787
{`To build out identical resources in other environments, create and provision a workflow using the below template.`}{' '}
8888
<EuiLink href={CREATE_WORKFLOW_LINK} target="_blank">
8989
Learn more

public/pages/workflow_detail/resizable_workspace.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@ export function ResizableWorkspace(props: ResizableWorkspaceProps) {
276276
titleSize="s"
277277
body={
278278
<>
279-
<EuiText>
279+
<EuiText size="s">
280280
Only valid workflows created from this OpenSearch Dashboards
281281
application are editable and viewable.
282282
</EuiText>

public/pages/workflow_detail/tools/errors/errors.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ export function Errors(props: ErrorsProps) {
1919
return (
2020
<>
2121
{isEmpty(props.errorMessage) ? (
22-
<EuiText>There are no errors.</EuiText>
22+
<EuiText size="s">There are no errors.</EuiText>
2323
) : (
2424
<EuiCodeBlock fontSize="m" isCopyable={false}>
2525
{props.errorMessage}

public/pages/workflow_detail/tools/resources/resources.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ export function Resources(props: ResourcesProps) {
4040
titleSize="s"
4141
body={
4242
<>
43-
<EuiText>
43+
<EuiText size="s">
4444
Provision the workflow to generate resources in order to start
4545
prototyping.
4646
</EuiText>

public/pages/workflow_detail/tools/tools.tsx

+3-3
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ import {
1313
EuiPanel,
1414
EuiTab,
1515
EuiTabs,
16-
EuiTitle,
16+
EuiText,
1717
} from '@elastic/eui';
1818
import { Workflow } from '../../../../common';
1919
import { Resources } from './resources';
@@ -117,9 +117,9 @@ export function Tools(props: ToolsProps) {
117117
}}
118118
>
119119
<EuiFlexItem grow={false}>
120-
<EuiTitle>
120+
<EuiText size="s">
121121
<h2>Tools</h2>
122-
</EuiTitle>
122+
</EuiText>
123123
</EuiFlexItem>
124124
<EuiFlexItem grow={false}>
125125
<EuiTabs size="s" expand={false}>

public/pages/workflow_detail/workflow_inputs/ingest_inputs/ingest_data.tsx

+4-4
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
*/
55

66
import React from 'react';
7-
import { EuiFlexGroup, EuiFlexItem, EuiTitle } from '@elastic/eui';
7+
import { EuiFlexGroup, EuiFlexItem, EuiText } from '@elastic/eui';
88
import { TextField } from '../input_fields';
99
import { AdvancedSettings } from './advanced_settings';
1010

@@ -17,9 +17,9 @@ export function IngestData(props: IngestDataProps) {
1717
return (
1818
<EuiFlexGroup direction="column">
1919
<EuiFlexItem grow={false}>
20-
<EuiTitle size="s">
21-
<h2>Ingest data</h2>
22-
</EuiTitle>
20+
<EuiText size="s">
21+
<h3>Ingest data</h3>
22+
</EuiText>
2323
</EuiFlexItem>
2424
<EuiFlexItem>
2525
<TextField label="Index name" fieldPath={'ingest.index.name'} />

public/pages/workflow_detail/workflow_inputs/ingest_inputs/source_data.tsx

+6-5
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ import {
1818
EuiModalHeaderTitle,
1919
EuiSpacer,
2020
EuiText,
21-
EuiTitle,
2221
EuiFilterGroup,
2322
EuiSmallFilterButton,
2423
EuiSuperSelectOption,
@@ -253,7 +252,9 @@ export function SourceData(props: SourceDataProps) {
253252
(option) =>
254253
({
255254
value: option.name,
256-
inputDisplay: <EuiText>{option.name}</EuiText>,
255+
inputDisplay: (
256+
<EuiText size="s">{option.name}</EuiText>
257+
),
257258
disabled: false,
258259
} as EuiSuperSelectOption<string>)
259260
)}
@@ -288,9 +289,9 @@ export function SourceData(props: SourceDataProps) {
288289
)}
289290
<EuiFlexGroup direction="column" gutterSize="s">
290291
<EuiFlexItem grow={false}>
291-
<EuiTitle size="s">
292-
<h2>Import data</h2>
293-
</EuiTitle>
292+
<EuiText size="s">
293+
<h3>Source data</h3>
294+
</EuiText>
294295
</EuiFlexItem>
295296
{props.lastIngested !== undefined && (
296297
<EuiFlexItem grow={false}>

public/pages/workflow_detail/workflow_inputs/processor_inputs/modals/configure_prompt_modal.tsx

+3-1
Original file line numberDiff line numberDiff line change
@@ -115,6 +115,7 @@ export function ConfigurePromptModal(props: ConfigurePromptModalProps) {
115115
anchorPosition="downLeft"
116116
>
117117
<EuiContextMenu
118+
size="s"
118119
initialPanelId={0}
119120
panels={[
120121
{
@@ -140,7 +141,7 @@ export function ConfigurePromptModal(props: ConfigurePromptModalProps) {
140141
/>
141142
</EuiPopover>
142143
<EuiSpacer size="m" />
143-
<EuiText>Prompt</EuiText>
144+
<EuiText size="s">Prompt</EuiText>
144145
<EuiSpacer size="s" />
145146
<EuiCodeEditor
146147
mode="json"
@@ -203,6 +204,7 @@ export function ConfigurePromptModal(props: ConfigurePromptModalProps) {
203204
<EuiPopover
204205
isOpen={schemaPopoverOpen}
205206
closePopover={() => setSchemaPopoverOpen(false)}
207+
panelPaddingSize="s"
206208
button={
207209
<EuiSmallButtonEmpty
208210
onClick={() =>

public/pages/workflow_detail/workflow_inputs/processor_inputs/modals/input_transform_modal.tsx

+11-8
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ import {
2626
EuiCodeBlock,
2727
EuiPopoverTitle,
2828
EuiIconTip,
29-
EuiSwitch,
29+
EuiCompressedSwitch,
3030
EuiCallOut,
3131
} from '@elastic/eui';
3232
import {
@@ -281,7 +281,7 @@ export function InputTransformModal(props: InputTransformModalProps) {
281281
<EuiSpacer size="s" />
282282
</>
283283
)}
284-
<EuiText>Source input</EuiText>
284+
<EuiText size="s">Source input</EuiText>
285285
<EuiSmallButton
286286
style={{ width: '100px' }}
287287
isLoading={isFetching}
@@ -436,7 +436,7 @@ export function InputTransformModal(props: InputTransformModalProps) {
436436
</EuiFlexItem>
437437
<EuiFlexItem>
438438
<>
439-
<EuiText>Define transform</EuiText>
439+
<EuiText size="s">Define transform</EuiText>
440440
<EuiSpacer size="s" />
441441
<MapArrayField
442442
field={props.inputMapField}
@@ -496,10 +496,12 @@ export function InputTransformModal(props: InputTransformModalProps) {
496496
)}
497497
<EuiFlexItem grow={true}>
498498
{transformOptions.length <= 1 ? (
499-
<EuiText>Transformed input</EuiText>
499+
<EuiText size="s">Transformed input</EuiText>
500500
) : (
501501
<EuiCompressedSelect
502-
prepend={<EuiText>Transformed input for</EuiText>}
502+
prepend={
503+
<EuiText size="s">Transformed input for</EuiText>
504+
}
503505
options={transformOptions}
504506
value={selectedTransformOption}
505507
onChange={(e) => {
@@ -513,6 +515,7 @@ export function InputTransformModal(props: InputTransformModalProps) {
513515
<EuiPopover
514516
isOpen={popoverOpen}
515517
closePopover={() => setPopoverOpen(false)}
518+
panelPaddingSize="s"
516519
button={
517520
<EuiSmallButtonEmpty
518521
onClick={() => setPopoverOpen(!popoverOpen)}
@@ -562,10 +565,10 @@ export function InputTransformModal(props: InputTransformModalProps) {
562565
<>
563566
<EuiFlexGroup direction="row">
564567
<EuiFlexItem grow={false}>
565-
<EuiText>Transformed prompt</EuiText>
568+
<EuiText size="s">Transformed prompt</EuiText>
566569
</EuiFlexItem>
567570
<EuiFlexItem grow={false} style={{ marginTop: '16px' }}>
568-
<EuiSwitch
571+
<EuiCompressedSwitch
569572
label="Show"
570573
checked={viewPromptDetails}
571574
onChange={() => setViewPromptDetails(!viewPromptDetails)}
@@ -583,7 +586,7 @@ export function InputTransformModal(props: InputTransformModalProps) {
583586
{viewPromptDetails && (
584587
<>
585588
<EuiSpacer size="s" />
586-
<EuiSwitch
589+
<EuiCompressedSwitch
587590
label="With transformed inputs"
588591
checked={viewTransformedPrompt}
589592
onChange={() =>

public/pages/workflow_detail/workflow_inputs/processor_inputs/modals/output_transform_modal.tsx

+5-4
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,7 @@ export function OutputTransformModal(props: OutputTransformModalProps) {
203203
)}
204204
<EuiFlexGroup direction="row" justifyContent="spaceBetween">
205205
<EuiFlexItem>
206-
<EuiText>Source output</EuiText>
206+
<EuiText size="s">Source output</EuiText>
207207
</EuiFlexItem>
208208
{!isEmpty(
209209
parseModelOutputsObj(props.modelInterface, fullResponsePath)
@@ -212,6 +212,7 @@ export function OutputTransformModal(props: OutputTransformModalProps) {
212212
<EuiPopover
213213
isOpen={popoverOpen}
214214
closePopover={() => setPopoverOpen(false)}
215+
panelPaddingSize="s"
215216
button={
216217
<EuiSmallButtonEmpty
217218
onClick={() => setPopoverOpen(!popoverOpen)}
@@ -382,7 +383,7 @@ export function OutputTransformModal(props: OutputTransformModalProps) {
382383
</EuiFlexItem>
383384
<EuiFlexItem>
384385
<>
385-
<EuiText>Define transform</EuiText>
386+
<EuiText size="s">Define transform</EuiText>
386387
<EuiSpacer size="s" />
387388
<MapArrayField
388389
field={props.outputMapField}
@@ -418,10 +419,10 @@ export function OutputTransformModal(props: OutputTransformModalProps) {
418419
<EuiFlexItem>
419420
<>
420421
{transformOptions.length <= 1 ? (
421-
<EuiText>Transformed output</EuiText>
422+
<EuiText size="s">Transformed output</EuiText>
422423
) : (
423424
<EuiCompressedSelect
424-
prepend={<EuiText>Transformed output for</EuiText>}
425+
prepend={<EuiText size="s">Transformed output for</EuiText>}
425426
options={transformOptions}
426427
value={selectedTransformOption}
427428
onChange={(e) => {

public/pages/workflow_detail/workflow_inputs/processor_inputs/modals/override_query_modal.tsx

+1
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,7 @@ export function OverrideQueryModal(props: OverrideQueryModalProps) {
112112
anchorPosition="downLeft"
113113
>
114114
<EuiContextMenu
115+
size="s"
115116
initialPanelId={0}
116117
panels={[
117118
{

public/pages/workflow_detail/workflow_inputs/processors_list.tsx

+1
Original file line numberDiff line numberDiff line change
@@ -204,6 +204,7 @@ export function ProcessorsList(props: ProcessorsListProps) {
204204
anchorPosition="downLeft"
205205
>
206206
<EuiContextMenu
207+
size="s"
207208
initialPanelId={PANEL_ID}
208209
panels={[
209210
{

public/pages/workflow_detail/workflow_inputs/search_inputs/configure_search_request.tsx

+4-5
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ import {
1515
EuiCompressedSuperSelect,
1616
EuiSuperSelectOption,
1717
EuiText,
18-
EuiTitle,
1918
EuiSpacer,
2019
} from '@elastic/eui';
2120
import {
@@ -83,9 +82,9 @@ export function ConfigureSearchRequest(props: ConfigureSearchRequestProps) {
8382
)}
8483
<EuiFlexGroup direction="column">
8584
<EuiFlexItem grow={false}>
86-
<EuiTitle size="s">
87-
<h2>Configure query</h2>
88-
</EuiTitle>
85+
<EuiText size="s">
86+
<h3>Configure query</h3>
87+
</EuiText>
8988
</EuiFlexItem>
9089
<EuiFlexItem grow={false}>
9190
<EuiCompressedFormRow label="Retrieval index">
@@ -100,7 +99,7 @@ export function ConfigureSearchRequest(props: ConfigureSearchRequestProps) {
10099
(option) =>
101100
({
102101
value: option.name,
103-
inputDisplay: <EuiText>{option.name}</EuiText>,
102+
inputDisplay: <EuiText size="s">{option.name}</EuiText>,
104103
disabled: false,
105104
} as EuiSuperSelectOption<string>)
106105
)}

public/pages/workflow_detail/workflow_inputs/search_inputs/edit_query_modal.tsx

+1
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,7 @@ export function EditQueryModal(props: EditQueryModalProps) {
6666
anchorPosition="downLeft"
6767
>
6868
<EuiContextMenu
69+
size="s"
6970
initialPanelId={0}
7071
panels={[
7172
{

0 commit comments

Comments
 (0)