Skip to content

Vega Custom Toolbar - #23

Closed
b-yogesh wants to merge 7 commits into
mainfrom
yogesh-vega-styling
Closed

Vega Custom Toolbar#23
b-yogesh wants to merge 7 commits into
mainfrom
yogesh-vega-styling

Conversation

@b-yogesh

Copy link
Copy Markdown
Collaborator

No description provided.

@b-yogeshb-yogesh self-assigned this Oct 28, 2024

export interface DashiDropdownProps extends Omit<DropdownState, "type"> {
onPropertyChange: PropertyChangeHandler;
panelIndex: number;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The term panel does not exist is the API, the lib part. This is relevant fort the demo only.

Also, I don't see why this is needed in every component now.

@forman

Copy link
Copy Markdown
Member

@b-yogesh I close this PR, I create an issue instead that specifies what is needed.

@b-yogesh

Copy link
Copy Markdown
CollaboratorAuthor

@b-yogesh I close this PR, I create an issue instead that specifies what is needed.

Sure! Let's do that. When we start working on it, we use this PR as a starting point to get some ideas (if needed)

@formanforman mentioned this pull request Mar 13, 2025
4 tasks
@forman

Copy link
Copy Markdown
Member

@b-yogesh I close this PR, I create an issue instead that specifies what is needed.

See #111

@formanforman closed this Mar 14, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@b-yogesh@forman