diff --git a/CHANGELOG.md b/CHANGELOG.md index 5ebf8ae3f797..6ad257196f78 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -68,6 +68,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) - [Table Visualization] Remove custom styling for text-align:center in favor of OUI utility class. ([#4164](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/4164)) - Migrate from legacy elasticsearch client to opensearch-js client in `osd-opensearch-archiver` package([#4142](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/4142)) - Replace the use of `bluebird` in `saved_objects` plugin ([#4026](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/4026)) +- [VisBuilder] Use OUI icon ([#4446](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/4446)) - [Vis Colors] Replace color maps with OUI color palettes ([#4293](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/4293)) - [Vis Colors] [Maps] Replace hardcoded color to OUI color in `maps_legacy` plugin ([#4294](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/4294)) - [Vis Colors] [TSVB] Update default color in `vis_type_timeseries` to use `ouiPaletteColorBlind()[0]`([#4363](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/4363)) diff --git a/src/plugins/vis_builder/public/assets/vis_builder_icon.svg b/src/plugins/vis_builder/public/assets/vis_builder_icon.svg deleted file mode 100644 index 69da8016eb12..000000000000 --- a/src/plugins/vis_builder/public/assets/vis_builder_icon.svg +++ /dev/null @@ -1,25 +0,0 @@ - - - - - - - - - - - - - - - - - - - - - - - - - diff --git a/src/plugins/vis_builder/public/assets/vis_builder_icon_secondary_fill.svg b/src/plugins/vis_builder/public/assets/vis_builder_icon_secondary_fill.svg deleted file mode 100644 index cdaad42f6276..000000000000 --- a/src/plugins/vis_builder/public/assets/vis_builder_icon_secondary_fill.svg +++ /dev/null @@ -1,25 +0,0 @@ - - - - - - - - - - - - - - - - - - - - - - - - - diff --git a/src/plugins/vis_builder/public/embeddable/vis_builder_embeddable_factory.tsx b/src/plugins/vis_builder/public/embeddable/vis_builder_embeddable_factory.tsx index 90048ba91322..3c0bf0337369 100644 --- a/src/plugins/vis_builder/public/embeddable/vis_builder_embeddable_factory.tsx +++ b/src/plugins/vis_builder/public/embeddable/vis_builder_embeddable_factory.tsx @@ -26,7 +26,6 @@ import { VisBuilderOutput, VISBUILDER_EMBEDDABLE, } from './vis_builder_embeddable'; -import visBuilderIcon from '../assets/vis_builder_icon.svg'; import { getStateFromSavedObject } from '../saved_visualizations/transforms'; import { getHttp, @@ -55,7 +54,7 @@ export class VisBuilderEmbeddableFactory name: PLUGIN_NAME, includeFields: ['visualizationState'], type: VISBUILDER_SAVED_OBJECT, - getIconForSavedObject: () => visBuilderIcon, + getIconForSavedObject: () => 'visBuilder', }; // TODO: Would it be better to explicitly declare start service dependencies? diff --git a/src/plugins/vis_builder/public/plugin.ts b/src/plugins/vis_builder/public/plugin.ts index 1445de923010..4e8f020d1fe8 100644 --- a/src/plugins/vis_builder/public/plugin.ts +++ b/src/plugins/vis_builder/public/plugin.ts @@ -24,8 +24,6 @@ import { VisBuilderStart, } from './types'; import { VisBuilderEmbeddableFactory, VISBUILDER_EMBEDDABLE } from './embeddable'; -import visBuilderIconSecondaryFill from './assets/vis_builder_icon_secondary_fill.svg'; -import visBuilderIcon from './assets/vis_builder_icon.svg'; import { EDIT_PATH, PLUGIN_ID, @@ -189,7 +187,7 @@ export class VisBuilderPlugin description: i18n.translate('visBuilder.visPicker.description', { defaultMessage: 'Create visualizations using the new VisBuilder', }), - icon: visBuilderIconSecondaryFill, + icon: 'visBuilder', stage: 'experimental', aliasApp: PLUGIN_ID, aliasPath: '#/', @@ -200,7 +198,7 @@ export class VisBuilderPlugin description: attributes?.description, editApp: PLUGIN_ID, editUrl: `${EDIT_PATH}/${encodeURIComponent(id)}`, - icon: visBuilderIcon, + icon: 'visBuilder', id, savedObjectType: VISBUILDER_SAVED_OBJECT, stage: 'experimental',