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

[Request] Option to set default state of paint panel & tool #12

Closed
Biont opened this issue Apr 26, 2020 · 6 comments · Fixed by #30
Closed

[Request] Option to set default state of paint panel & tool #12

Biont opened this issue Apr 26, 2020 · 6 comments · Fixed by #30
Labels
enhancement New feature or request

Comments

@Biont
Copy link

Biont commented Apr 26, 2020

Hello again!

This tool keeps putting a smile on my face every workday and I would like to thank you again for creating it.

My main use case is highlighting specific parts of a screenshot - because I'd use grim/slurp directly if it wasn't. Therefore, I almost always need to bring up the paint panel (then select rectangle/arrow because I don't want to annoy people with shitty mouse drawings). It would be pretty cool if I could force-enable the paint panel in the config file - and probably even the default tool selection.

What do you think about adding such a feature?

@jtheoof
Copy link
Owner

jtheoof commented May 17, 2020

That's a fair request @Biont. Have you considered also the keyboard shortcuts in the meantime? A for arrow, R for rectangle, ... I never toggle the panel unless I need to play with the color of the drawing.

@Biont
Copy link
Author

Biont commented May 18, 2020

Sure I use them and I admit that I am asking for something rather trivial which probably only adds up to a slight inconvenience during a regular work day.
It's still a feature that pops up in my head whenever I have to go bring up my desired panel. I cannot tell how much hassle it would be to implement this though, so I will absolutely be happy with whatever you decide

@jtheoof
Copy link
Owner

jtheoof commented May 30, 2020

I was thinking. How about a persist_state option? That way you find swappy UI in the same state you were in last time. Would that work better or you'd still prefer a default tool selection? I'm just thinking of ideas.

@jtheoof jtheoof added the enhancement New feature or request label May 30, 2020
@Biont
Copy link
Author

Biont commented May 31, 2020

I believe that would work reasonably well. Another option would be command line arguments/flags to launch swappy in the desired state.

jtheoof added a commit that referenced this issue Jun 18, 2020
Allows to toggle paint panel on/off upon startup.

Closes #12
jtheoof added a commit that referenced this issue Jun 18, 2020
Allows to toggle paint panel on/off upon startup.

Closes #12
@jtheoof
Copy link
Owner

jtheoof commented Jun 18, 2020

Hi @Biont so I've added a show_panel setting in the config file.

With this part in your ~/.config/swappy/config:

[Default]
...
show_panel=true
...

You'll be able to have the panel on when starting swappy.

Regarding the mode as CLI input params, I'm not convinced yet. This is mainly a UI tool, if I add more painting modes, it will become a little harder to maintain.

I'll wait to see if more users ask for this feature.

@Biont
Copy link
Author

Biont commented Jun 18, 2020

Awesome, thank you so much

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants