6 TWTRTimelineViewController issues

[AceGreen1989]

@chaselatta I have created a demo project to show issue #1, #2, #4, #6 in detail. Ignore other issues for now. issues are listed in order of importance.

https://dl.dropboxusercontent.com/u/8061535/twitter.zip

Please follow these steps to replicate each of the 4 issues.

Issue #1:

• Run project on iPhone (from small to large)
• Rotate between landscape and portrait
• Look for white backgrounds going over tweets (landscape mode)
  *** See them mostly on iPad and in landscape mode
• Look for tweets with pictures with large space above and below picture (landscape)
  *** The larger the screen, the bigger the space (like on iPad) happens only in landscape

Issue #2:

• Editor -> Embed In -> Tab Bar Controller
• Select the tab Bar item and set the System Item to whatever
• Run project = image will not appear
• Use solution provided and put a UINavigationController in between the two as such
  UITabBarController -> UINavigationController -> UITableViewController
• Item will appear
• Item would appear if you didn’t use TWTRTimelineViewController as the class for the UITableViewController

Issue #4:

• With current setup (i.e. UITabBarController -> UINavigationController -> UITableViewController)
• Give your UITableViewController a title and add a barButtonItems (to the navigationBar provided by the UINavigationController)
• Run project, nothing will appear
• Title and barbuttonitems would appear if you didn’t use TWTRTimelineViewController as the class for the UITableViewController

Issue #6:

• Add a .swift file for your UITabBarController and assign your Controller to it

• In your .swift file create a variable called symbol

• In your UITableViewController, create the same symbol variable.

• Try to do the following in your UITableViewController class

  let parentTabBarController = self.tabBarController as! ChartDetailTabBarController
  symbol = parentTabBarController.symbol

• Above code would work if your UITableViewController class was not TWTRTimelineViewController

• Regardless if you have a UINavigationController between your UITableViewController and your UITabBarController

[chaselatta]

@AceGreen1989 we have looked into the issues and found a bug that was causing all the issues except for number 1. The problem stems from how we are initializing code when the view controller is created from interface builder. We have implemented a fix that will be going out either today or tomorrow. If you want to work around this for now you can just manually the create the view controller in code which will avoid the problem.

As far as issue #1, we are looking into that right now and will hopefully have more info soon.

[AceGreen1989]

Hey, you just made me super happy!!! Awesome stuff

Regarding issue #1: can you replicate the issue. if not maybe we can have a short chat to show you how it looks on my iPad. It would be lovely to solve this issue while you are at it.

A side request: Can you add the ability to change the backgroundColor of TWTRTimelineViewController. Currently its some shade of gray.

[chaselatta]

just call self.view.backgroundColor = myColor in your viewDidLoad method or wherever it makes sense for you to do so.

We are able to reproduce issue number 1 but have not had a chance to implement a fix.

[AceGreen1989]

Yea that was known but I’m using IB and was wondering why we can’t just set it there? I wonder if this will be fixed with the new initialization you are implementing.

Please note issue #1 is the most important one. I would really love for it to be included in the next SDK release so I can submit my app for iOS9

[AceGreen1989]

@chaselatta

Following feedback based on Twitter 1.10.0

Issue #1:
Still open and probably the most important one

New issues:

Issue #7: View won’t appear unless scrolled
The TWTRTimelineViewController seems to load the data but not present it as before. Now I need to tap/scroll somewhere within the view for it to reflect. Was not like that before, and nothing in my code changed

Issue #8: Setting backgroundColor:
Setting backgroundColor through IB is not possible (should be). Setting up backgroundColor through self.view.backgroundColor = UIColor.whiteColor() causes the separators to appear without data (as a standard UITableView would without data). Previously and without setting backgroundColor, the empty view would just appear grey without separators. Also in this case, the view needs tap/scroll before data appears

Additional comment:
I would like to add is regarding the twitter icon in each tableViewCell. In my opinion thats a little cheesy. I do understand that twitter want to show that its their tweets and some sort of brand recognition but maybe it should be done differently. It only makes sense to add an icon on tweet cell if we are available to add other steaming sources and you want to show the twitters coming from twitter.

Solution: I would suggest placing the icon somewhere else and in one place. For example, the refresh can be a twitter icon instead of the animating circle, etc.

[chaselatta]

@AceGreen1989

We did not ship a fix for issue number 1 in version 1.10.0; this bug has been prioritized and we will try to get a fix in place for our next release. However, you should be able to work around this bug by calling [self.tableView beginUpdates]; [self.tableView endUpdates]; during your rotation events - this will refresh the height of the cells. The exact time you need to call this depends on which version of iOS you are targeting because the rotation methods all changed in iOS 8.

We will look into issue number 7 and try to get a fix out in the next release but for the time being can you try calling reloadData on the tableview when the data loads to force a refresh?

The problem with issue 8 is that we are setting the background color in viewDidLoad. I have updated the code already to honor the value set in IB and that change will go out in the next release.

Regarding the Twitter Icon. This is something that is not likely to change as this was a decision made by our design and branding teams. If you want more information about this decision you can contact support@fabric.io.

[AceGreen1989]

Regarding issue #1:

I’m only targeting iOS 8 & 9 and I tried the following

        self.tableView.beginUpdates()
        
        // Request Twitter Feed
        let client = TWTRAPIClient()
        self.dataSource = TWTRSearchTimelineDataSource(searchQuery: "$\(self.symbol)", APIClient: client)
        
        self.tableView.endUpdates()

This doesn’t fix the height issues. the height issues are not because of rotation. As a matter of fact, my app is only available in landscape. This issue is there when the View is loaded. I tried to move the code above into the ViewWillLayoutSubviews() but that caused infinite amount of messages saying there is no guest session and that I should that deprecated loginGuest method.

Issue #7: reload data after self.dataSource doesn’t help as there is no completion handler now so there is no way to check if its done before reloading table.

Also if I wait long enough (LONG) the data does load (without tap/scroll) with the following error:

2015-09-11 18:27:36.918 StockSwipe[994:191850] This application is modifying the autolayout engine from a background thread, which can lead to engine corruption and weird crashes. This will cause an exception in a future release.
Stack:(
0 CoreFoundation 0x00000001834d8f74 + 148
1 libobjc.A.dylib 0x0000000197f7ff80 objc_exception_throw + 56
2 CoreFoundation 0x00000001834d8ea4 + 0
3 Foundation 0x00000001844f25d8 + 88
4 Foundation 0x0000000184374a1c + 36
5 UIKit 0x0000000188b33958 + 64
6 UIKit 0x0000000188b33b9c + 548
7 UIKit 0x0000000188b33968 + 80
8 UIKit 0x0000000188a2a2d8 + 240
9 UIKit 0x0000000188b7335c + 640

10  UIKit                               0x0000000188b306d4 <redacted> + 68
11  UIKit                               0x0000000188b372e8 <redacted> + 28
12  UIKit                               0x0000000188b3a420 <redacted> + 364
13  StockSwipe                          0x00000001002c7428 +[TWTRViewUtil addVisualConstraints:toView:options:metrics:views:] + 168
14  StockSwipe                          0x00000001002c7248 +[TWTRViewUtil addVisualConstraints:toView:views:] + 92
15  StockSwipe                          0x00000001002b7024 -[TWTRTweetTableViewCell commonInit] + 420
16  StockSwipe                          0x00000001002b6e6c -[TWTRTweetTableViewCell initWithStyle:reuseIdentifier:] + 84
17  UIKit                               0x0000000188b772ec <redacted> + 500
18  StockSwipe                          0x00000001002a1434 -[TWTRTimelineViewController tableView:cellForRowAtIndexPath:] + 100
19  UIKit                               0x0000000188d7167c <redacted> + 688
20  UIKit                               0x0000000188d717d4 <redacted> + 80
21  UIKit                               0x0000000188d613d4 <redacted> + 2440
22  UIKit                               0x0000000188d76364 <redacted> + 104
23  UIKit                               0x0000000188b17a90 <redacted> + 176
24  UIKit                               0x0000000188a2b00c <redacted> + 644
25  QuartzCore                          0x0000000188231f14 <redacted> + 148
26  QuartzCore                          0x000000018822cb20 <redacted> + 292
27  QuartzCore                          0x000000018822c9e0 <redacted> + 32
28  QuartzCore                          0x000000018822c07c <redacted> + 252
29  QuartzCore                          0x000000018822bdd0 <redacted> + 516
30  QuartzCore                          0x000000018825af48 <redacted> + 236
31  libsystem_pthread.dylib             0x00000001989821e8 <redacted> + 584
32  libsystem_pthread.dylib             0x0000000198981d60 <redacted> + 136
33  libsystem_pthread.dylib             0x0000000198981544 pthread_mutex_lock + 0
34  libsystem_pthread.dylib             0x0000000198981028 start_wqthread + 4

)

issue #8: Happy to hear that.

Twitter icon: I’ll email them my comments below.

[AceGreen1989]

Just took 1.10.1 for a spin, here’s some feedback:

Issue #1: still there
Issue #7: resolved
Issue #8: still there

[AceGreen1989]

Hello,

Just installed the latest update,

Issue #1: Still there
Issue #8: Resolved

Regarding issue #8: I did notice that now as data is being loaded, the table shows the standard separators. this looks ugly in my opinion. I did extend the footer to cover it (thats the current workout with regular UITableViews)

Just an idea: if you guys add something like https://github.com/dzenbot/DZNEmptyDataSet you can use it for branding twitter and removing the icon from every tweet. That view appears when the view first appears, and as long as their is no data (you can set when it should appear. also you can add the reload to be a twitter icon.

Also there is no a way to check for no data and data done loading

heres how the view looks in my app:

[AceGreen1989]

Status report: 1.11.1

Issue #1: Still there

Also see comments I made regarding issue #8 because they are still apply

[MDias9]

Has this issue been resolved? I’ve been running into problem #5 listed above, where the navigation bar does not appear for a UIViewController. Embedding the ViewController in a NavigationController didn’t solve the problem.

[AceGreen1989]

Status report: 1.11.2

Issue #1: Still there

[AceGreen1989]

Any progress? Patiently waiting here

[SeeNewcastleUK]

Can’t wait any longer for a solution.

Have removed the Twitter Tab and Fabric from my App until a solution is found to Issue #1

Richard

[AceGreen1989]

Its a little frustrating that its taking so long, I shipped an update with issue #1 in place.

I’m surprised not more developers are pushing this but that’s why its not at a high priority for Fabric to fix.

[esteluk]

Well, it’s the most popular non-Bitcode topic here!

For the sake of completeness, this issue is still present with 1.11.3

[AceGreen1989]

You know guys, I have submitted an application for a position at Twitter. if you guys hire me, I promise my first order of business will be to solve issue #1

[AceGreen1989]

As pointed out by a twitter dev, my screenshots are no longer valid. Dropbox disables links after a certain period. Just shows how long these issues have been open

[niall]