Possibility to deselect assets when displaying picker for second time.

[krodak]

Added possibility to deselect assets when showing picker multiple times.

If DKImagePickerController is used in some VC multiple time it might be useful sometimes to clear selection on each use. Currently, if I want to show clean selection I'd need to create new instance of image picker, which is not memory efficient.

Use case:
There is VC to add new images with some custom titles, etc to 'posts feed'. User can choose maximum X posts, but he can display picker multiple time. As each time all selected photos are used in application VC, picker should show no selection each time it's shown.

[krodak]

@zhangao0086 I'm not sure whether this is the best approach, but I needed this one for my project.

[zhangao0086]

Thanks @krodak !
I already understand your needs and it's necessary function!!!
I've made a new commit that's trying to solve the problem:0e23cd6. Maybe its better :)

[krodak]

@zhangao0086 I see advantage of your approach but if I understand correctly, programmer needs to clear selection each time he shows image picker.
With mine approach you can just set it per instance - do you think this approach could be taken to yours? :-) Usually, when you create image picker for the use in specific VC and case, it will be used in consistent way, so those kind of settings would be nice to have per instance.

[zhangao0086]

@krodak
Yes, you're right. That case is very common. But I prefer to make a subclass like this:

public class DKMomentaryImagePickerController: DKImagePickerController {

    /// This allows to clear @selectedAssets without creating new picker each time
    public var shouldDeselectAssets: Bool = false

    override public func viewWillAppear(animated: Bool) {
        super.viewWillAppear(animated)

        if shouldDeselectAssets {
            self.defaultSelectedAssets = nil
        }
    }

}

It will be quite easy to expand and not depend on the specific scene. What do you think?

[krodak]

Sounds great!
On Thu, 4 Feb 2016 at 23:32, Bannings [email protected] wrote:

@krodak https://github.com/krodak
Yes, you're right. That case is very common. But I prefer to make a
subclass like this:

public class DKMomentaryImagePickerController: DKImagePickerController {

/// This allows to clear @selectedAssets without creating new picker each time
public var shouldDeselectAssets: Bool = false

override public func viewWillAppear(animated: Bool) {
    super.viewWillAppear(animated)

    if shouldDeselectAssets {
        self.defaultSelectedAssets = nil
    }
}

}

It will be quite easy to expand and not depend on the specific scene. What
do you think?

—
Reply to this email directly or view it on GitHub
#69 (comment)
.