Skip to content
This repository has been archived by the owner on Jan 19, 2023. It is now read-only.

Add property CustomSvg to icons and navigation #2786

Merged
merged 1 commit into from
Sep 8, 2021

Conversation

ftovaro
Copy link
Contributor

@ftovaro ftovaro commented Aug 20, 2021

What this PR does / why we need it:

This allows plugin authors to setup their own personal SVG for their plugins

Screen Shot 2021-08-23 at 11 02 48 AM

Which issue(s) this PR fixes

Special notes for your reviewer:

Release note:

release-note

@ftovaro ftovaro force-pushed the issue-1422 branch 10 times, most recently from 3701e53 to 5066163 Compare August 23, 2021 17:44
@ftovaro ftovaro marked this pull request as ready for review August 23, 2021 19:32
pkg/navigation/navigation.go Outdated Show resolved Hide resolved
web/src/stories/icon.stories.mdx Outdated Show resolved Hide resolved
web/src/stories/icon.stories.mdx Show resolved Hide resolved
web/src/stories/docs/plugins/GoPluginsIntro.story.mdx Outdated Show resolved Hide resolved
@ftovaro ftovaro force-pushed the issue-1422 branch 12 times, most recently from 2954a80 to 44c272f Compare August 30, 2021 17:02
@@ -53,6 +53,32 @@ export const iconWithTooltipView= {
}
};

export const svg = `
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The validation might be too aggressive. This is valid input currently being filtered out as seen in storybook

Signed-off-by: ftovaro <[email protected]>
Signed-off-by: Vikram Yadav <[email protected]>
@GuessWhoSamFoo GuessWhoSamFoo merged commit d4f47ac into vmware-archive:master Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow plugins to specify a custom icon
3 participants