Skip to content

added macOS support and removed any View in CachedAsyncImage View#1

Open
iankoex wants to merge 1 commit intobullinnyc:mainfrom
iankoex:run-on-macOS
Open

added macOS support and removed any View in CachedAsyncImage View#1
iankoex wants to merge 1 commit intobullinnyc:mainfrom
iankoex:run-on-macOS

Conversation

@iankoex
Copy link
Copy Markdown

@iankoex iankoex commented Oct 20, 2023

I have added support for macOS by using typealias for NSImage and UIImage.
I have also removed any View and using generics for placeholder, image and error views.

@bullinnyc
Copy link
Copy Markdown
Owner

@iankoex Hi! macOS support is very good, I'll be glad to add it. But I specifically did not use a generic in order to preserve the ability to use different constructors, for example, an error or placeholder parameter is not always needed.

@iankoex
Copy link
Copy Markdown
Author

iankoex commented Oct 21, 2023

That's alright.
Tests on macOS don't pass because of the way image is loaded from Bundle.module.

@bullinnyc bullinnyc force-pushed the main branch 6 times, most recently from 7a65b56 to a5e0a6e Compare October 23, 2023 14:05
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