Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create a PluginAction for toggling connection #178

Closed
wants to merge 1 commit into from
Closed

Create a PluginAction for toggling connection #178

wants to merge 1 commit into from

Conversation

vocksel
Copy link

@vocksel vocksel commented May 31, 2019

Add's a user-configurable shortcut to start/stop connections to the Rojo server.

image

Had to hoist up some state from ConnectPanel which I would rather have not done, but not sure if there's a better way without Rodux.

@LPGhatguy
Copy link
Contributor

Hey, this fixes #103, too!

@LPGhatguy
Copy link
Contributor

Finally getting a chance to look at this!

Mechanically, this change looks pretty much perfect. I'm not super sure about the use case for toggling the connection with a hotkey though! What makes you want to do this kind of thing, and is there something else we can fix that might be more direct?

@vocksel
Copy link
Author

vocksel commented Jun 3, 2019

My biggest motivation is the workflow of opening the ribbon bar, clicking the plugin button, then moving the mouse to select the connect button is tedious. I know that yall are looking to have multiple connections tho, so I understand a single "connect" may not be entirely possible anymore.

If the plugin had an icon so I could cleanly have it on the quick access bar without having the "Connect" text there, that would help a lot, and could be a nice alternative if you don't want to use plugin actions.

@vocksel
Copy link
Author

vocksel commented Jun 20, 2020

This PR is well over a year old and I don't see this workflow as necessary. Gonna go ahead and close it down so it's not floating around anymore.

@vocksel vocksel closed this Jun 20, 2020
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.

2 participants