diff --git a/CHANGELOG.md b/CHANGELOG.md index 200a8e653248..7f1fe53fea1c 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -133,6 +133,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) - [Dynamic Configurations] Fix dynamic config API calls to pass correct input ([#6474](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/6474)) - [BUG][Multiple Datasource] Modify the button of selectable component to fix the title overflow issue ([#6465](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/6465)) - [BUG][Multiple Datasource] Validation succeed as long as status code in response is 200 ([#6399](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/6399)) +- [BUG][Multiple Datasource] Fix on data source selectable and readonly component are not consistent ([#6545]https://github.com/opensearch-project/OpenSearch-Dashboards/pull/6545) - [BUG][Multiple Datasource] Add validation for title length to be no longer than 32 characters [#6452](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/6452)) ### 🚞 Infrastructure diff --git a/config/opensearch_dashboards.yml b/config/opensearch_dashboards.yml index 40d643b014fd..853a785d26ce 100644 --- a/config/opensearch_dashboards.yml +++ b/config/opensearch_dashboards.yml @@ -247,7 +247,7 @@ # vis_builder.enabled: false # Set the value of this setting to true to enable multiple data source feature. -#data_source.enabled: false +data_source.enabled: true # Set the value of this setting to true to hide local cluster in data source feature. #data_source.hideLocalCluster: false # Set the value of these settings to customize crypto materials to encryption saved credentials diff --git a/src/plugins/data_source_management/public/components/data_source_selectable/__snapshots__/data_source_selectable.test.tsx.snap b/src/plugins/data_source_management/public/components/data_source_selectable/__snapshots__/data_source_selectable.test.tsx.snap index 79d158ee3be5..b3abb935afee 100644 --- a/src/plugins/data_source_management/public/components/data_source_selectable/__snapshots__/data_source_selectable.test.tsx.snap +++ b/src/plugins/data_source_management/public/components/data_source_selectable/__snapshots__/data_source_selectable.test.tsx.snap @@ -29,6 +29,9 @@ exports[`DataSourceSelectable should filter options if configured 1`] = ` ownFocus={true} panelPaddingSize="none" > + - - - + - - - + - - -
+
+
+
+ DATA SOURCES + ( + 3 + ) +
+
+
+ +
+
+
-
-
- DATA SOURCES - ( - 3 - ) -
-
-
- -
-
-
-
-
- +
+ class="euiFormControlLayoutCustomIcon" + > + +
diff --git a/src/plugins/data_source_management/public/components/data_source_selectable/data_source_selectable.tsx b/src/plugins/data_source_management/public/components/data_source_selectable/data_source_selectable.tsx index 2259f93fbbec..c1aafb0b2a30 100644 --- a/src/plugins/data_source_management/public/components/data_source_selectable/data_source_selectable.tsx +++ b/src/plugins/data_source_management/public/components/data_source_selectable/data_source_selectable.tsx @@ -11,8 +11,7 @@ import { EuiPanel, EuiButtonEmpty, EuiSelectable, - EuiSpacer, - EuiHorizontalRule, + EuiPopoverTitle, } from '@elastic/eui'; import { ApplicationStart, @@ -270,14 +269,16 @@ export class DataSourceSelectable extends React.Component< anchorPosition="downLeft" data-test-subj={'dataSourceSelectableContextMenuPopover'} > + - - - - + {(list, search) => ( <> - {search} + {search} {list} )} diff --git a/src/plugins/data_source_management/public/components/data_source_view/__snapshots__/data_source_view.test.tsx.snap b/src/plugins/data_source_management/public/components/data_source_view/__snapshots__/data_source_view.test.tsx.snap index 954e49247957..efe373e7d825 100644 --- a/src/plugins/data_source_management/public/components/data_source_view/__snapshots__/data_source_view.test.tsx.snap +++ b/src/plugins/data_source_management/public/components/data_source_view/__snapshots__/data_source_view.test.tsx.snap @@ -24,24 +24,18 @@ exports[`DataSourceView Should render successfully when provided datasource has ownFocus={true} panelPaddingSize="none" > + - - - + - - -