mirror of
https://github.com/PlayCover/PlayCover.git
synced 2026-04-26 00:05:52 +03:00
[PR #308] [MERGED] Fix no app icon for some app #1567
Labels
No labels
UI
app-support
bug
bug
documentation
duplicate
enhancement
game-support
good first issue
help wanted / caution
inactive
invalid
macos-beta
pull-request
question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/PlayCover#1567
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/PlayCover/PlayCover/pull/308
Author: @KhoraLee
Created: 9/6/2022
Status: ✅ Merged
Merged: 9/7/2022
Merged by: @JoseMoreville
Base:
develop← Head:fix/appIcon📝 Commits (2)
e422f9dfeat: Extract app icon from assets.car64fa7d6fix: Load icon also from assets.car to fix no icon📊 Changes
5 files changed (+137 additions, -14 deletions)
View changed files
📝
PlayCover.xcodeproj/project.pbxproj(+14 -0)📝
PlayCover/Model/PlayApp.swift(+25 -14)📝
PlayCover/PlayCover-Bridging-Header.h(+1 -0)➕
PlayCover/Utils/AssetsExtractor.swift(+75 -0)➕
PlayCover/Utils/CoreUI.h(+22 -0)📄 Description
This PR will extract app icon's also from
Assets.carand select highest resolution icon.Still apps that doesn't have app icon will show spinning progress circle.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.
KeymapViewVM.swiftfile #1924