opened
piv199
username_0
Also Removed code duplicate

#217
fixed bug setting image on undefined thread for `imageWithURL` related methods
issue
created
lfarah
username_1
Hmmmm, I read the method ```imageWithUrl(url:)``` and I'm thinking about how this naming can be better in Swift 3. Maybe do an initializer like ```UIImageView(url: String)```

What are your thoughts?
comment
created
piv199
username_0
@username_1 looks awesome, but what's about supporting storyboards?
comment
created
lfarah
username_1
I didn't see any @IBDesignable or @IBInspectable... How is this extension supporting storyboards?
comment
created
piv199
username_0
@username_1 imagine you set up storyboard view controller with view and UIImageView on it, then you set reference to this UIImageView in your ViewController class. Once ViewController viewDidLoad you would like to load an image into UIImageView from url.

You cannot reassign `UIImageView` with `UIImageView(url: String)`, but you can do `imageView.imageWithURL(...)`
comment
created
lfarah
username_1
Alright. Now I understood what you meant by "supporting storyboards".
What if we add this initializer and also change the name to something like:```imageView.image(urlString:...)```, so it looks more like Swift 3
comment
created
piv199
username_0
@username_1 yes it makes sense, also I think there are should be support for urlString and url. About naming, yes, I haven't changed this file yet
comment
created
piv199
username_0
@username_1 this is just hot fix, naming is about another pull request, also adding new features is out of scope current pr
comment