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

UI tweak and Linux command updates #1179

Merged
merged 3 commits into from
Mar 5, 2025
Merged

Conversation

Mbucari
Copy link
Collaborator

@Mbucari Mbucari commented Mar 5, 2025

Move encoding options above to the mp3 settings column (#764)

I saw that @CLHatch recommend relocating the encoding options, and I thought it made sense.

The Audio File settings now look like this:

image
image

I also added some more tool tips for some of the more cryptic options.

Use polkit (#1176) and dnf5/dnf (#1177)

@rmcrackan
Copy link
Owner

Nice. I like it. Thank you again.

@rmcrackan rmcrackan merged commit bf87180 into rmcrackan:master Mar 5, 2025
9 checks passed
@Mbucari
Copy link
Collaborator Author

Mbucari commented Mar 5, 2025

You're very welcome. Just a reminder, today is my last day for a while. Probably until the 19th.

@@ -25,21 +25,59 @@ public LinuxInterop(params object[] values) { }
public void SetFolderIcon(string image, string directory) => throw new PlatformNotSupportedException();
public void DeleteFolderIcon(string directory) => throw new PlatformNotSupportedException();

public string ReleaseIdString => LibationScaffolding.ReleaseIdentifier.ToString() + (File.Exists("/bin/yum") ? "_RPM" : "");
public string ReleaseIdString => LibationScaffolding.ReleaseIdentifier.ToString() + (File.Exists("/bin/apt") ? "_DEB" : "_RPM");

This comment was marked as resolved.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rmcrackan
Copy link
Owner

You're very welcome. Just a reminder, today is my last day for a while. Probably until the 19th.

Have a great vacation!

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