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

Update files under dist to refer to Azahar instead of Citra #738

Merged
merged 1 commit into from
Mar 21, 2025

Conversation

HurricanePootis
Copy link
Contributor

Currently, the mime file and icon are named citra.xml and citra.svg respectively. Furthermore, the actual MIME's are x-ctr-*. Therefore, I am proposing that (at least on Linux), to move over to using azahar.xml and azahar.svg, along side changing the MIME's to x-ctr-*.

For me at least, one reason for doing this is that whenever using Papirus icons, whenever the azahar.desktop file specifies Icon=citra, instead of showing azahar's logo from the citra.svg installed with the emulator, it show's Papirus' version of Citra's icon. Another reason as to consider this is to further differentiate this emulator from what came before.

One thing this PR does not do is change the icon shown in the About menu, as I do not know how to do that as of right now.

I am open to feedback and comments!

Images

Before PR:
image
image
After PR:
image
image

@OpenSauce04
Copy link
Member

The mimetypes shouldn't be changed. ctr refers to the codename of the 3DS, not to Citra. Citra was named after the ctr codename.

@HurricanePootis
Copy link
Contributor Author

@OpenSauce04 Done

@HurricanePootis
Copy link
Contributor Author

HurricanePootis commented Mar 20, 2025

There is one problem I am having with these that I hope I could get some understanding. Without the patch notes, the orange blossom does indeed show in the about section:
image
But, with the patch, it does this:
image

It's kind of funny, ngl, but I would like to know why. I am not a QT person at all, so I am going to look through and try my best. But, I'd figure I'd post here and see if anybody reading this knows why.

@OpenSauce04 OpenSauce04 changed the title Switch icon name, mime type, and desktop file to use Azahar naming Change icon and desktop file names to refer to Azahar Mar 20, 2025
@OpenSauce04 OpenSauce04 self-requested a review March 20, 2025 11:14
@OpenSauce04 OpenSauce04 self-assigned this Mar 20, 2025
@pull-request-size pull-request-size bot added size/M and removed size/S labels Mar 21, 2025
@OpenSauce04 OpenSauce04 changed the title Change icon and desktop file names to refer to Azahar Update files under dist to refer to Azahar instead of Citra Mar 21, 2025
@OpenSauce04
Copy link
Member

OpenSauce04 commented Mar 21, 2025

Testing checklist:

  • Linux
  • MacOS
  • Windows

This resolves some icon theming issues on Linux

Co-authored-by: HurricanePootis <[email protected]>
@OpenSauce04 OpenSauce04 merged commit b225e85 into azahar-emu:master Mar 21, 2025
12 checks passed
@HurricanePootis
Copy link
Contributor Author

@OpenSauce04 Thank you for your hard work!

@HurricanePootis HurricanePootis deleted the dist branch March 22, 2025 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants