Skip to content

hide/show connect/disconnect instead of just disabling#5

Open
SoMuchForSubtlety wants to merge 1 commit into
mattn:mainfrom
SoMuchForSubtlety:connect-disconnect-toggle
Open

hide/show connect/disconnect instead of just disabling#5
SoMuchForSubtlety wants to merge 1 commit into
mattn:mainfrom
SoMuchForSubtlety:connect-disconnect-toggle

Conversation

@SoMuchForSubtlety

@SoMuchForSubtlety SoMuchForSubtlety commented Oct 13, 2021

Copy link
Copy Markdown
Contributor

image
image

@mattn

mattn commented Oct 13, 2021

Copy link
Copy Markdown
Owner

Hmm, Those are always appears on Windows ver.
image

@SoMuchForSubtlety

Copy link
Copy Markdown
Contributor Author

Hmm, Those are always appears on Windows ver.
image

looks like it's also like that for mac
image

To me it makes more sense to only show the option you can actually use, but I guess it is better to be consistent with the other versions.

@SoMuchForSubtlety

SoMuchForSubtlety commented Oct 13, 2021

Copy link
Copy Markdown
Contributor Author

Its also worth considering, that the connect and disconnect options take up less space on Windows and mac than on Linux (at least with gnome).

@mattn

mattn commented Oct 13, 2021

Copy link
Copy Markdown
Owner

Give me a while to decide which is easier to use. :)

@SoMuchForSubtlety

Copy link
Copy Markdown
Contributor Author

I just realized, if you're not connected windows also only shows one option 😄
image

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