Skip to content
This repository has been archived by the owner on Oct 30, 2018. It is now read-only.

Adds support for an optional view to host outlets under the text view #562

Merged
merged 3 commits into from
Jan 10, 2017

Conversation

dzenbot
Copy link

@dzenbot dzenbot commented Jan 7, 2017

The blue view is just an example of what could be added down there.
image

Ignacio Romero Zurbuchen added 2 commits January 6, 2017 13:40
…, adjusting its height based on its subviews. Non-visible by default. Subviews' layout should be configured using auto-layout as well.
@@ -2410,38 +2413,8 @@ - (void)didReceiveMemoryWarning
- (void)dealloc
{
[self slk_unregisterNotifications];

_tableView.delegate = nil;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yay!

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, this was completely unnecessary and legacy. Pre-ARC bad practice 🤦

@dzenbot dzenbot merged commit ac17d97 into master Jan 10, 2017
@dzenbot dzenbot deleted the bottom-content-view-support branch January 10, 2017 03:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants