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

Update troubleshooting instructions #127

Closed
wants to merge 1 commit into from
Closed

Update troubleshooting instructions #127

wants to merge 1 commit into from

Conversation

DenisGorbachev
Copy link

No description provided.

@hiroshi
Copy link
Contributor

hiroshi commented Jan 31, 2019

Thanks for your contribution.

However, disabling the open browser functionality may cause some troubles (see #109).

So we should add some notes for this section.

NOTE: Disabling the open browser functionality may cause some troubles (see https://github.com/gyazo/Gyazo-for-Linux/pull/109).

We'll add this line before/after merging this pull request.

@pastak
Copy link
Member

pastak commented Jan 31, 2019

@DenisGorbachev Would you tell us why you add this troubleshooting instructions.

I'm not agree to write it on README because this confuses user. I have no idea that this instruction works.

@hiroshi
Copy link
Contributor

hiroshi commented Feb 1, 2019

I can understand some people don't want to open images in a browser tab/window just after uploading.
Official Windows version of Gyazo client has this option in preferences for those.

Yep, but it should not be in the troubleshooting section of README I think. New sections like Tips or HowTo can be more suitable, or somewhere else I'm not sure.

@ghost
Copy link

ghost commented Aug 11, 2019

I'm unable to find the .gyazo.config.yml file from $HOME/.gyazo.config.yml to add the:

browser_cmd: echo

code to prevent the web browser pop up. Any suggestions?

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