Skip to content
This repository has been archived by the owner on Apr 6, 2021. It is now read-only.

moved user/tabs/@show to state/tabs/@show #238

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bmatusiak
Copy link
Contributor

moved user/tabs/@show to state/tabs/@show

**** as this could be different from another workspace

moved user/tabs/@show to state/tabs/@show

**** as this could be different from another workspace
["title", "false"],
["asterisk", "false"]
]);
settings.setDefaults("state/tabs", []);
settings.setDefaults("state/tabs", [ ]);
Copy link
Member

Choose a reason for hiding this comment

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

Did you mean to move ["show", "true"], here?

@nightwing
Copy link
Member

Do you use tabs/@show preference, or need it for another plugin?

@bmatusiak
Copy link
Contributor Author

I tried to put it there on line 203, and it kept getting over written.

And yes my popup plugin now sets this to false, and it takes over to my other workspaces.

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.

2 participants