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

Mimes: Use 'applications-other' for executables #432

Merged
merged 1 commit into from
Mar 15, 2024

Conversation

newhoa
Copy link
Contributor

@newhoa newhoa commented Mar 3, 2024

Symlink the Mime icon application-x-executable to the Category icon applications-other. It uses the same design as what is used currently but has better contrast, looks better on dark themes, and is used more widely in the theme to represent applications.

This will change the icon used for executable and binary mimes, as well as some executable-related actions like "Create Launcher..." on Xfdesktop right click.


The applications-other icon is used for:

  • "All Applications" in Whisker Menu
  • "Launcher" Xfce panel plugin
  • "Applications" when right clicking on Xfdesktop

The white-square-with-gear is also used for .desktop files on Xfdesktop and thunar, and also more closely matches the icon used for other executable mimes like Appimage, Debs, RPMs, etc.

Symlink the Mime icon `application-x-executable` to the
Category icon `applications-other`. It uses the same design
as what is used currently but has better contrast, looks better
on dark themes, and is used more widely in the theme to represent applications.

This will change the icon used for executable and binary mimes, as well as some
executable-related actions like "Create Launcher..." on Xfdesktop right click.

---

The `applications-other` icon is used for:
- "All Applications" in Whisker Menu
- "Launcher" Xfce panel plugin
- "Applications" when right clicking on Xfdesktop

The white-square-with-gear is also used for .desktop files on Xfdesktop
and thunar, and also more closely matches the icon used for other executable
mimes like Appimage, Debs, RPMs, etc.
@bluesabre
Copy link
Member

I think this is an improvement. @ochosi, any concerns?

@ochosi
Copy link
Member

ochosi commented Mar 8, 2024

No concerns from my side!

@newhoa newhoa merged commit 885c9af into shimmerproject:master Mar 15, 2024
1 check passed
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