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

MX-Linux - Updated icons for mx-apps #3424

Merged
merged 1 commit into from
May 12, 2023

Conversation

MelberGithub
Copy link
Contributor

Hello PapirusDevelopmentTeam,

I am a member of the dev team at MX-Linux.

As you (hopefully) know, papirus is the default icon theme for MX-Linux. In preparation for the upcoming MX-23 release we decided to update some of the icons for our mx-apps and attempt to bring them more in line with general papirus styling.

Many of these icons are already included in papirus, so I have made a pull request for the following updated icons:

  • GPG-Key4
  • idevice
  • mx-conky
  • mx-datetime
  • mx-live-usb-maker
  • mx-packageinstaller
  • mx-snapshot
  • mx-tweak
  • mx-user

We would appreciate it very much if you could update these icons, so that MX-23 has a consistent look on release.

Thank you for all your great work.

Cheers, Melber

Edited icons:

GPG-Key4
idevice
mx-conky
mx-datetime
mx-live-usb-maker
mx-packageinstaller
mx-snapshot
mx-tweak
mx-user
@SmartFinn
Copy link
Member

Thanks for the contribution. Most of the icons have several minor issues (e.g. differ baseline). I'll merge it when I have sure that I can fix it.

@MelberGithub
Copy link
Contributor Author

Hi @SmartFinn,
thanks for the quick response.

I hope there are no major problems with the icons, I followed the guidelines to the best of my ability. Let me know if something really needs to be redone.

Cheers and stay safe,
Melber

@SmartFinn SmartFinn merged commit 69cee06 into PapirusDevelopmentTeam:master May 12, 2023
SmartFinn added a commit that referenced this pull request May 12, 2023
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