De-couple 'enable autoshow' from command-shift-c shortcut. #44

Closed
heeton opened this Issue May 1, 2012 · 3 comments

Comments

Projects
None yet
3 participants
@heeton

heeton commented May 1, 2012

I'd like to be able to show/hide the output box, and not have it auto-show whenever guard pops out some new output.

Can we de-couple the 'enable autoshow' from the command-shift-c shortcut?

@cyphactor

This comment has been minimized.

Show comment
Hide comment
@cyphactor

cyphactor May 1, 2012

Owner

@heeton, please explain in more detail your ideal workflow and the reasoning behind you wanting this change as it will better guide me in providing an optimal solution. Also, explain why you think de-coupling the 'enable autoshow' would improve that workflow.

Owner

cyphactor commented May 1, 2012

@heeton, please explain in more detail your ideal workflow and the reasoning behind you wanting this change as it will better guide me in providing an optimal solution. Also, explain why you think de-coupling the 'enable autoshow' would improve that workflow.

@whitequark

This comment has been minimized.

Show comment
Hide comment
@whitequark

whitequark Jul 25, 2012

Contributor

@cyphactor, I understand what's the reasoning behind this feature request. When I just write my code (and save it after each few lines), I don't care about guard and the pane constantly popping out annoys me. On the other hand, when I fix some bug, I need it. In whatever case, I don't want to fiddle with the command palette: it takes too much time and attention.

I fixed this by adding an extra command and a keybinding to just run tests and show their output without enabling autoshow.

Contributor

whitequark commented Jul 25, 2012

@cyphactor, I understand what's the reasoning behind this feature request. When I just write my code (and save it after each few lines), I don't care about guard and the pane constantly popping out annoys me. On the other hand, when I fix some bug, I need it. In whatever case, I don't want to fiddle with the command palette: it takes too much time and attention.

I fixed this by adding an extra command and a keybinding to just run tests and show their output without enabling autoshow.

@cyphactor

This comment has been minimized.

Show comment
Hide comment
@cyphactor

cyphactor Jul 25, 2012

Owner

I have pulled in pull request #53 and cleaned it up a bit. Therefore, I am closing this issues as there is now a way to run the test and show the Guard output without enabling auto show.

Owner

cyphactor commented Jul 25, 2012

I have pulled in pull request #53 and cleaned it up a bit. Therefore, I am closing this issues as there is now a way to run the test and show the Guard output without enabling auto show.

@cyphactor cyphactor closed this Jul 25, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment