Skip to content

feat: set default archive app in default applications page#1860

Open
thegabriele97 wants to merge 3 commits into
pop-os:masterfrom
thegabriele97:feat_default_archive_app
Open

feat: set default archive app in default applications page#1860
thegabriele97 wants to merge 3 commits into
pop-os:masterfrom
thegabriele97:feat_default_archive_app

Conversation

@thegabriele97
Copy link
Copy Markdown

PR supporting pop-os/cosmic-files#1555


  • I have disclosed use of any AI generated code in my commit messages.
    • If you are using an LLM, and do not fully understand the changes it is making to the code base, do not create a PR.
    • In our experience, AI generated code often results in overly complex code that lacks enough context for a proper fix or feature inclusion. This results in considerably longer code reviews. Due to this, AI authored or partially authored PRs may be closed without comment.
  • I understand these changes in full and will be able to respond to review comments.
  • My change is accurately described in the commit message.
  • My contribution is tested and working as described.
  • I have read the Developer Certificate of Origin and certify my contribution under its conditions.

@thegabriele97 thegabriele97 force-pushed the feat_default_archive_app branch from 990d41d to 5892e6b Compare February 19, 2026 20:41
@jacobgkau jacobgkau requested review from a team February 19, 2026 21:16
@jacobgkau jacobgkau self-assigned this Feb 19, 2026
@thegabriele97 thegabriele97 force-pushed the feat_default_archive_app branch from 5892e6b to a9b243b Compare March 26, 2026 08:44
@jacobgkau jacobgkau force-pushed the feat_default_archive_app branch 2 times, most recently from 171d5ac to 91d10ed Compare April 2, 2026 22:06
thegabriele97 and others added 2 commits April 7, 2026 14:04
Required after the iced 0.14 rebase in libcosmic.
@jacobgkau jacobgkau force-pushed the feat_default_archive_app branch from 91d10ed to c100de3 Compare April 7, 2026 20:04
wash2
wash2 previously approved these changes Apr 7, 2026
Copy link
Copy Markdown
Member

@jacobgkau jacobgkau left a comment

Choose a reason for hiding this comment

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

The sole remaining problem with this is that, on upgrade, Settings shows COSMIC Files as the default application, but both Firefox and COSMIC Files actually treat File Roller as the default. If I change the default application to File Roller and then back to COSMIC Files, then they respect COSMIC Files as the default. The desync needs to be fixed and the default needs to actually be COSMIC Files on upgrade.

@thegabriele97
Copy link
Copy Markdown
Author

the default needs to actually be COSMIC Files on upgrade.

Mmmh how can this be achieved? I mean, "forcing" cosmic files to be the default in all the supported archive mime types on upgrade

@thegabriele97
Copy link
Copy Markdown
Author

I have a question about how Cosmic Settings handles multiple MIME types.

If I set "Cosmic Video Player" as the default under the Video category, and then manually change a specific MIME type (for example, video/x-matroska) to use mpv, Cosmic Video Player still appears as the default app for Videos in the settings.

It seems like the same issue occurs with the new Archive category: Settings shows Cosmic Files as the default application, even when a specific MIME type is actually being opened with Firefox.

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