Skip to content

Mimeapp GUI plugin - #342

Merged
ItsLemmy merged 9 commits into
noctalia-dev:mainfrom
Mathew-D:plugin/mimeapp-gui
Mar 25, 2026
Merged

Mimeapp GUI plugin#342
ItsLemmy merged 9 commits into
noctalia-dev:mainfrom
Mathew-D:plugin/mimeapp-gui

Conversation

@Mathew-D

@Mathew-D Mathew-D commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

This plugin uses a Python script to scan .desktop files on your system, then lets you change which application is assigned to each MIME type by updating ~/.config/mimeapps.list.
I broke the mime types up into groups for each changing.

@Mathew-D

Mathew-D commented Mar 9, 2026

Copy link
Copy Markdown
Contributor Author

Should solve: noctalia-dev/noctalia#1903

@Mathew-D
Mathew-D marked this pull request as draft March 10, 2026 22:13
-tried to do grouping of basic types but doesn't work yet
@Mathew-D Mathew-D closed this Mar 10, 2026
@ceviches

Copy link
Copy Markdown

that's so good, i got noctalia today and was just looking for something like that! how does one add this plugin? it doesn't show up in my available plugins list even though i already updated it

@Mathew-D Mathew-D reopened this Mar 11, 2026
@Mathew-D

Copy link
Copy Markdown
Contributor Author

Updated the gui on the Common page to allow easier change for some. Left the rest for more advance users. Added an IPC so users can create a .desktop if they don't want the icon on the bar.

@Mathew-D
Mathew-D marked this pull request as ready for review March 11, 2026 01:44
@Mathew-D

Copy link
Copy Markdown
Contributor Author

@ceviches when PR is accepted it will show up in the plugin list. If you want to use the plugin before you must download the folder and put it into your plugin folder which is: ~/.config/noctalia/plugins/

@spiros132 spiros132 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Some general feedback, questions and comment I have about this PR :)

Comment thread mimeapp-gui/BarWidget.qml Outdated
Comment thread mimeapp-gui/Main.qml Outdated
Comment thread mimeapp-gui/manifest.json Outdated
Comment thread mimeapp-gui/README.md Outdated
Comment thread mimeapp-gui/Panel.qml Outdated
Comment thread mimeapp-gui/Panel.qml Outdated
Comment thread mimeapp-gui/Panel.qml Outdated
Comment thread mimeapp-gui/Panel.qml Outdated
Comment thread mimeapp-gui/Panel.qml Outdated
@Mathew-D

Copy link
Copy Markdown
Contributor Author

Thank you for the feedback, I am going away for a week or 2 and will look at your ideas when I get back.

@ItsLemmy
ItsLemmy marked this pull request as draft March 14, 2026 15:13
…lback literals

- Added more i18n keys in en.json
- Updated statusMessage and error handling to use i18n keys throughout
- Removed duplicate toggle() IPC handler from Main.qml
- Fixed a minor typo in README.md and updated IPC usage documentation
@github-actions

Copy link
Copy Markdown
Contributor

Automatic Code Quality Review

File: mimeapp-gui/BarWidget.qml

  • (L) Line 16: The applyUiScale: false would make it so that the component does not support ui scaling. Always check if this is the correct behaviour you want when changing the ui scale!
+  applyUiScale: false

File: mimeapp-gui/Main.qml

File: mimeapp-gui/Panel.qml

@Mathew-D
Mathew-D marked this pull request as ready for review March 25, 2026 20:55

@spiros132 spiros132 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Some very minor things otherwise ready :)

Comment thread mimeapp-gui/Panel.qml Outdated
Comment thread mimeapp-gui/Panel.qml Outdated
Added logic to use capitalized group key as tab name if translation is missing
Ensures user-friendly tab names even without i18n entry
 Uses string interpolation instead of concatenation
@github-actions

Copy link
Copy Markdown
Contributor

Automatic Code Quality Review

File: mimeapp-gui/BarWidget.qml

  • (L) Line 16: The applyUiScale: false would make it so that the component does not support ui scaling. Always check if this is the correct behaviour you want when changing the ui scale!
+  applyUiScale: false

File: mimeapp-gui/Main.qml

File: mimeapp-gui/Panel.qml

@spiros132

Copy link
Copy Markdown
Collaborator

@ItsLemmy

@github-actions

Copy link
Copy Markdown
Contributor

Automatic Code Quality Review

File: mimeapp-gui/BarWidget.qml

  • (L) Line 16: The applyUiScale: false would make it so that the component does not support ui scaling. Always check if this is the correct behaviour you want when changing the ui scale!
+  applyUiScale: false

File: mimeapp-gui/Main.qml

File: mimeapp-gui/Panel.qml

@ItsLemmy

Copy link
Copy Markdown
Contributor

Ignore the + applyUiScale: false bot review. This should be false for a barWidget.

@Mathew-D

Copy link
Copy Markdown
Contributor Author

Can it be merge then.

@ItsLemmy

Copy link
Copy Markdown
Contributor

If you remove the official: false in the manifest, sure.

@ItsLemmy
ItsLemmy merged commit 4af40b9 into noctalia-dev:main Mar 25, 2026
1 check passed
@ItsLemmy

Copy link
Copy Markdown
Contributor

Thanks

@Mathew-D
Mathew-D deleted the plugin/mimeapp-gui branch April 16, 2026 23:16
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.

4 participants