Skip to content

Allow users to set activeTabset to undefined#326

Open
alexwaeseperlman wants to merge 2 commits into
caplin:masterfrom
alexwaeseperlman:master
Open

Allow users to set activeTabset to undefined#326
alexwaeseperlman wants to merge 2 commits into
caplin:masterfrom
alexwaeseperlman:master

Conversation

@alexwaeseperlman

Copy link
Copy Markdown

This PR creates a way to not have a selected tab. I needed this feature for a project I'm working on with popout windows and I couldn't find another way to do it.

@alexwaeseperlman

Copy link
Copy Markdown
Author

Just checking in on the status of this PR

@david-trumid

david-trumid commented Jul 13, 2023

Copy link
Copy Markdown

Just checking in on the status of this PR

can you elaborate on the usage of not needing this? I am interested in something similiar cc @alexwaeseperlman

@alexwaeseperlman

Copy link
Copy Markdown
Author

For that project I ended up using my own fork

@alexwaeseperlman

Copy link
Copy Markdown
Author

Hi this has been open for a while. Is there a reason why it was never reviewed?

@nealus

nealus commented Sep 17, 2023

Copy link
Copy Markdown
Collaborator

Hi Alex, I think changing Actions.setActiveTabset to accept undefined would be better?

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.

3 participants