-
Notifications
You must be signed in to change notification settings - Fork 57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Provide a way to override brick cell content after the updateContent #58
Provide a way to override brick cell content after the updateContent #58
Conversation
wfsttam
commented
Jan 6, 2017
- provides a means of showing placeholder content
@@ -24,6 +24,16 @@ public protocol BrickCellTapDelegate: UIGestureRecognizerDelegate { | |||
func didTapBrickCell(brickCell: BrickCell) | |||
} | |||
|
|||
public protocol OverrideContentSource: class { | |||
func overrideContent(forBrickCell brickCell: BrickCell) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about the resetCell
function to clean up the placeholder views?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
@@ -24,6 +24,16 @@ public protocol BrickCellTapDelegate: UIGestureRecognizerDelegate { | |||
func didTapBrickCell(brickCell: BrickCell) | |||
} | |||
|
|||
public protocol OverrideContentSource: class { | |||
func overrideContent(forBrickCell brickCell: BrickCell) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's do it "swift 3"-style
func overrideContent(for cell: BrickCell)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
f8966a0
to
7c71342
Compare
7c71342
to
a572e99
Compare
…k cell - provides a way to show content while content is not available yet - added tests - example included in 'DynamicContentViewController'
a572e99
to
74c7ec8
Compare
} | ||
|
||
extension OverrideContentSource { | ||
public func overrideContent(for brickCell: BrickCell) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this optional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
Current coverage is 93.65% (diff: 100%)
|
Adds ability to insert temporary content or override content into a brick cell after updateContent is called: Fixes #46 |