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

Use macOS SD-Card icon instead of the generic one #21

Merged
merged 1 commit into from
Feb 16, 2021
Merged

Use macOS SD-Card icon instead of the generic one #21

merged 1 commit into from
Feb 16, 2021

Conversation

ffosilva
Copy link

This pull request sets the same icon used by Apple SD card reader kext for the SDDisk IOService.

Screen Shot 2021-02-13 at 20 23 27

@cholonam
Copy link
Owner

Nice. I'll merge and upload a new release as soon as I have some time. Thanks.

@ffosilva
Copy link
Author

Take your time! Thank you very much for your effort in making this kext as best as possible!

@ffosilva ffosilva marked this pull request as draft February 14, 2021 13:36
@ffosilva
Copy link
Author

I converted it to draft because I'll try to externalize the icon configuration into Info.plist.

@ffosilva ffosilva marked this pull request as ready for review February 14, 2021 16:07
@ffosilva
Copy link
Author

I moved back to 'ready'. AFAIK there's no straightforward way to make this work by simply adding it as IOMediaIcon on Info.plist, but it should be done later (e.g. on a refactoring).

@cholonam cholonam merged commit 332048a into cholonam:master Feb 16, 2021
@cholonam
Copy link
Owner

I just applied your PR and uploaded a new release. Thanks again.

@ffosilva ffosilva deleted the set-icon branch February 20, 2021 16:19
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.

2 participants