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

Fix suru icon color #106

Closed
wants to merge 3 commits into from
Closed

Fix suru icon color #106

wants to merge 3 commits into from

Conversation

Feichtmeier
Copy link
Contributor

@franglais125 sorry for the mistake, now with the correct color!

Use white for minimize, maximize, close, restore buttons
For Yaru, Suru, Communitheme icon theme
All three themes are needed due to a name change for the communitheme snap and the new theme name for ubuntu 18.10

image

All three icon themes are the Suru icon theme, but to have support for 
all versions of the Communitheme installation there should be packages 
for all three.
For Yaru, Suru, Communitheme icon theme
All three themes are needed due to a name change for the communitheme snap and the new theme name for ubuntu 18.10 +
@franglais125
Copy link
Owner

Merged as [7e14934]. Thanks again!

@Feichtmeier
Copy link
Contributor Author

@franglais125 great thank you! Why did you close it and not merge it? :D maybe I lack some understanding here (really, no irony or something)

@franglais125
Copy link
Owner

I cherry-picked your new commit locally with git (not through github). When I pushed the changes, your code was already in and there was no need to merge this PR.

I often work more locally than through this interface!

@Feichtmeier
Copy link
Contributor Author

Ah thanks for clarification =) Best regards

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