Skip to content

Feature/#585 create base structure#586

Open
Ivanruii wants to merge 31 commits into
vnextfrom
feature/#585-create-base-structure
Open

Feature/#585 create base structure#586
Ivanruii wants to merge 31 commits into
vnextfrom
feature/#585-create-base-structure

Conversation

@Ivanruii
Copy link
Copy Markdown
Collaborator

Close #585

Ivanruii added 30 commits April 30, 2026 10:59
const map = new Map<string, any>();
flatOptions.forEach(flatOption => {
const { parentId, tabIndex, id, ...option } = flatOption;
const { tabIndex, id, ...option } = flatOption;
Copy link
Copy Markdown
Member

@nasdan nasdan May 12, 2026

Choose a reason for hiding this comment

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

Suggested change
const { tabIndex, id, ...option } = flatOption;
const { parentId: _parentId, tabIndex, id, ...option } = flatOption;

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Fixed

const map = new Map<string, any>();
flatOptions.forEach(flatOption => {
const { parentId, tabIndex, id, isSelectable, ...option } = flatOption;
const { tabIndex, id, isSelectable, ...option } = flatOption;
Copy link
Copy Markdown
Member

@nasdan nasdan May 12, 2026

Choose a reason for hiding this comment

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

Suggested change
const { tabIndex, id, isSelectable, ...option } = flatOption;
const { parentId: _parentId, tabIndex, id, isSelectable, ...option } = flatOption;

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Fixed

return undefined;
}
const { isSelectable, tabIndex, ...option } = internalSelectedOption;
const { ...option } = internalSelectedOption;
Copy link
Copy Markdown
Member

@nasdan nasdan May 12, 2026

Choose a reason for hiding this comment

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

Suggested change
const { ...option } = internalSelectedOption;
const { isSelectable: _isSelectable, tabIndex: _tabIndex, ...option } = internalSelectedOption;

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Fixed

Sign up for free to 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.

[VSCode Extension] Create base structure

2 participants