Skip to content

Commit

Permalink
Update data explorer ui to use compressed components (#8222)
Browse files Browse the repository at this point in the history
* update discover ui to use compressed components

Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>

* revert query editor changes

Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>

* Changeset file for PR #8222 created/updated

* update overflow prop for query editor

Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>

* update language selector snapshots

Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>

---------

Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
(cherry picked from commit 8d45530)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
  • Loading branch information
1 parent 97a792f commit 87e6276
Show file tree
Hide file tree
Showing 11 changed files with 159 additions and 170 deletions.
2 changes: 2 additions & 0 deletions changelogs/fragments/8222.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
fix:
- Update data explorer ui to use compressed components ([#8222](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/8222))
Original file line number Diff line number Diff line change
Expand Up @@ -3,15 +3,9 @@
* SPDX-License-Identifier: Apache-2.0
*/

import React from 'react';
import { EuiButtonIconProps, EuiSmallButtonIcon, EuiToolTip, EuiToolTipProps } from '@elastic/eui';
import { i18n } from '@osd/i18n';
import {
EuiSmallButtonIcon,
EuiButtonIconProps,
EuiText,
EuiToolTip,
EuiToolTipProps,
} from '@elastic/eui';
import React from 'react';

interface IDataSelectorRefresherProps {
tooltipText: string;
Expand All @@ -23,26 +17,24 @@ interface IDataSelectorRefresherProps {
export const DataSelectorRefresher: React.FC<IDataSelectorRefresherProps> = React.memo(
({ tooltipText, onRefresh, buttonProps, toolTipProps }) => {
return (
<EuiText size="s" className="sourceRefreshText">
<EuiToolTip
position="right"
content={i18n.translate('data.datasource.selector.refreshDataSources', {
defaultMessage: tooltipText,
})}
display="block"
data-test-subj="sourceRefreshButtonToolTip"
{...toolTipProps}
>
<EuiSmallButtonIcon
onClick={onRefresh}
iconType="refresh"
aria-label="sourceRefresh"
className="sourceRefreshButton"
data-test-subj="sourceRefreshButton"
{...buttonProps}
/>
</EuiToolTip>
</EuiText>
<EuiToolTip
position="right"
content={i18n.translate('data.datasource.selector.refreshDataSources', {
defaultMessage: tooltipText,
})}
display="block"
data-test-subj="sourceRefreshButtonToolTip"
{...toolTipProps}
>
<EuiSmallButtonIcon
onClick={onRefresh}
iconType="refresh"
aria-label="sourceRefresh"
className="sourceRefreshButton"
data-test-subj="sourceRefreshButton"
{...buttonProps}
/>
</EuiToolTip>
);
}
);
Original file line number Diff line number Diff line change
Expand Up @@ -3,17 +3,17 @@
* SPDX-License-Identifier: Apache-2.0
*/

import React, { useEffect, useCallback, useMemo } from 'react';
import { EuiCompressedComboBox } from '@elastic/eui';
import { i18n } from '@osd/i18n';
import { DataSource, DataSetWithDataSource, IndexPatternOption } from '../datasource';
import { DataSourceGroup, DataSourceOption, DataSourceSelectableProps } from './types';
import { DataSelectorRefresher } from './data_selector_refresher';
import React, { useCallback, useEffect, useMemo } from 'react';
import {
DATA_SELECTOR_DEFAULT_PLACEHOLDER,
DATA_SELECTOR_REFRESHER_POPOVER_TEXT,
DATA_SELECTOR_S3_DATA_SOURCE_GROUP_HINT_LABEL,
} from '../constants';
import { DataSetWithDataSource, DataSource, IndexPatternOption } from '../datasource';
import { DataSelectorRefresher } from './data_selector_refresher';
import { DataSourceGroup, DataSourceOption, DataSourceSelectableProps } from './types';

// Asynchronously retrieves and formats dataset from a given data source.
const getAndFormatDataSetFromDataSource = async (
Expand Down
10 changes: 5 additions & 5 deletions src/plugins/data/public/ui/dataset_selector/dataset_selector.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,23 +3,23 @@
* SPDX-License-Identifier: Apache-2.0
*/

import React, { useEffect, useMemo, useState, useCallback, useRef } from 'react';
import {
EuiButton,
EuiButtonEmpty,
EuiIcon,
EuiPopover,
EuiPopoverFooter,
EuiSelectable,
EuiSelectableOption,
EuiSmallButtonEmpty,
EuiToolTip,
} from '@elastic/eui';
import { FormattedMessage } from '@osd/i18n/react';
import React, { useCallback, useEffect, useMemo, useRef, useState } from 'react';
import { toMountPoint } from '../../../../opensearch_dashboards_react/public';
import { Dataset, DEFAULT_DATA } from '../../../common';
import { getQueryService } from '../../services';
import { IDataPluginServices } from '../../types';
import { AdvancedSelector } from './advanced_selector';
import { getQueryService } from '../../services';

interface DatasetSelectorProps {
selectedDataset?: Dataset;
Expand Down Expand Up @@ -133,15 +133,15 @@ export const DatasetSelector = ({
<EuiPopover
button={
<EuiToolTip content={`${selectedDataset?.title ?? 'Select data'}`}>
<EuiButtonEmpty
<EuiSmallButtonEmpty
className="datasetSelector__button"
iconType="arrowDown"
iconSide="right"
onClick={togglePopover}
>
<EuiIcon type={datasetIcon} className="datasetSelector__icon" />
{datasetTitle}
</EuiButtonEmpty>
</EuiSmallButtonEmpty>
</EuiToolTip>
}
isOpen={isOpen}
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 87e6276

Please sign in to comment.