Sol icon indicating copy to clipboard operation
Sol copied to clipboard

Replace UISearchDisplayController with UISearchController

Open tciuro opened this issue 10 years ago • 11 comments

Hello,

Now that Xcode 7 is out, it would be great to see Sol updated for iOS 8 or iOS 9. The UISearchDisplayController class been deprecated in iOS 8 with UISearchController, so when running Sol in iOS8 or later, the app doesn't work well (it takes several attempts to search and select the city to see the weather).

tciuro avatar Oct 10 '15 19:10 tciuro

Hello Comyar, I'm curious as to why you're not using Storyboard. Is it a matter of preference? Thanks for the app! :-)

tciuro avatar Oct 11 '15 02:10 tciuro

Yeah, really just preference. I never saw the advantage of using nibs and storyboards were really wonky when they first came out (i.e. impossible-to-resolve merge conflicts), so I just ended up writing my layout code by hand. Nowadays storyboards are much nicer, but old habits never die.

comyar avatar Oct 11 '15 05:10 comyar

Comyar, would you like to fix this problem as I was working on iOS8.4? Sol works no well to display weather info. I tried to change the change UISearchDisplayController to UISearchController and met other errors. :(

Samzhang029 avatar Apr 06 '16 07:04 Samzhang029

I'm currently writing a Swift version of the app (with an updated/cooler UI!) which is why I haven't gotten around to fixing this.

comyar avatar Apr 06 '16 15:04 comyar

I want to use this SOL into my automation testing demo. Now everything is OK except the display the weather. I am not good at the Objective-C, so cannot fix the problems. Do you have the github link of your Swift version of the app? If it can work, I can change to this app.

Samzhang029 avatar Apr 07 '16 00:04 Samzhang029

I don't have the Swift version on GitHub yet...are you saying you're having trouble getting location search results working? I'm not clear on what the issue is here.

On Apr 6, 2016, at 5:48 PM, SamZhang [email protected] wrote:

I want to use this SOL into my automation testing demo. Now everything is OK except the display the weather. I am not good at the Objective-C, so cannot fix the problems. Do you have the github link of your Swift version of the app? If it can work, I can change to this app.

— You are receiving this because you commented. Reply to this email directly or view it on GitHub

comyar avatar Apr 07 '16 01:04 comyar

Thanks for quick reply! No, app can search the locations.
The problem is after selected location it cannot display the weather information and it indicated an unhappy face and "Update Failed. Check your network connection." I am sure we have the internet access. Maybe it's not the problem of UISearchDisplayController. Do you like to have a try?

Samzhang029 avatar Apr 07 '16 01:04 Samzhang029

Yeah that's definitely not going to happen because of the UISearchDisplayController. Do you have a valid Wunderground API key?

On Apr 6, 2016, at 6:52 PM, SamZhang [email protected] wrote:

Thanks for quick reply! No, app can search the locations.

The problem is after selected location it cannot display the weather information and it indicated an unhappy face and "Update Failed. Check your network connection." I am sure we have the internet access. Maybe it's not the problem of UISearchDisplayController. Do you like to have a try?

— You are receiving this because you commented. Reply to this email directly or view it on GitHub

comyar avatar Apr 07 '16 01:04 comyar

Yes, we have a valid API_KEY. And if we use the web browser we can get the JSon. It's strange problem.

Samzhang029 avatar Apr 07 '16 02:04 Samzhang029

Hmm I'm not sure then...it should just work unless you're trying to add features, in which case it's likely just a bug on your end. I'd be happy to look at a code snippet or two if you think you can narrow down the problem to a few problem areas.

comyar avatar Apr 07 '16 04:04 comyar

Not bad at all

niarilorn avatar Feb 02 '24 17:02 niarilorn