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

Allow user to set icon size in settings #155

Closed
wants to merge 2 commits into from
Closed

Allow user to set icon size in settings #155

wants to merge 2 commits into from

Conversation

mswiszcz
Copy link

This PR allows user to set icon size in settings, making it easier to move from this:

before

to this:
after

Here's the new option:

newoption

May need some tweaking as I see the icon is not properly centered, not sure though if it's a problem with this or not related.

One commit is optional as Atom automatically removed unused whitespace for me.

Hope you like it 😄

@dscso
Copy link

dscso commented Oct 29, 2016

Please add this feature... I have a High DPI screen, and the icons are looking horrible... this fixes it.
Before:
bar
After:
screenshot-1
It would be better if the the icon size adjusts automatically like all the others icons do but I don't know how that is possible.
And, thank you all who made this applet.

@jaszhix
Copy link

jaszhix commented Oct 31, 2016

@mswiszcz I forked this project as Icing Task Manager and integrated your pull request as a part of the initial commit.

@mswiszcz
Copy link
Author

@jaszhix Good job, there is also Russian translation pending as PR on my repo, you may want to add this also 😉

@jaszhix
Copy link

jaszhix commented Oct 31, 2016

@mswiszcz Thanks, I'll look into the other PRs later this week! One caveat with the icon resizing, I had to restart Cinnamon to see the change. So mileage may vary. I'll dig into the API docs some more and investigate.

@jaszhix
Copy link

jaszhix commented Nov 1, 2016

@mswiszcz Russian translation was added.

@mswiszcz mswiszcz closed this Aug 16, 2017
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