Skip to content

OpenCanopy: Add SD card icon support - #657

Open
ilikesn0w wants to merge 1 commit into
acidanthera:masterfrom
ilikesn0w:OpenCanopySdCard
Open

ilikesn0w wants to merge 1 commit into
acidanthera:masterfrom
ilikesn0w:OpenCanopySdCard

Conversation

@ilikesn0w

@ilikesn0w ilikesn0w commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Added SD card icon support to OpenCanopy.

Regarding the changes to GuiClickableIsHit: Some icons may have transparent areas (for example, hole in the OpticalDrive or rectangular holes on the top sides of the SD card icon), and because of this, those areas cannot be pressed, my changes fixes this problem. Apple does this as well.

Linked PR: acidanthera/OcBinaryData#17

@ilikesn0w
ilikesn0w force-pushed the OpenCanopySdCard branch 4 times, most recently from 50b4c2b to a3babc6 Compare November 8, 2026 07:53
Comment on lines +439 to +447
if ( (DevicePathType (DevicePathWalker) == MESSAGING_DEVICE_PATH)
&& (DevicePathSubType (DevicePathWalker) == MSG_VENDOR_DP)
&& (DevicePathNodeLength (DevicePathWalker) >= sizeof (VENDOR_DEFINED_DEVICE_PATH)))
{
VendorNode = (VENDOR_DEFINED_DEVICE_PATH *)DevicePathWalker;
if (CompareGuid (&gAppleSdCardVendorDevicePathGuid, &VendorNode->Guid)) {
return TRUE;
}
}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Used some code from OcGetNextLoadOptionDevicePath here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant