Git Product home page Git Product logo

Comments (15)

ussjoin avatar ussjoin commented on May 14, 2024

Update for more information:
The first set of issues was with iPhone OS 3.0b4 AND iPhoneSDK 3.0. Recompiling under iPhoneSDK 2.2.1 (current stable release), the black cell problem disappears, but the userpics are still all not displayed.

from ioctocat.

dennisreimann avatar dennisreimann commented on May 14, 2024

The gravatars are loaded along with a user and cached on the device... hence you'll have to open a user detail view of someone who has a gravatar (the email address needs to be public and he must have an image at gravatar.com) to verify that it doesn't work.

The reason why they aren't loaded directly is: When the feed list gets loaded we just have the username but we need the email to fetch the gravatar. Loading all user details directly would result in too many API calls :(

from ioctocat.

ussjoin avatar ussjoin commented on May 14, 2024

I just checked, and even if I load a user detail view of myself (since I have a Gravatar), it still doesn't show my userpic, either on the user detail view, or on any of the other places in which it should appear. So the explanation still doesn't work-- it really is broken.

Also, while I get that we can't dynamically load gravatars on every load, surely you can load them in the list, and cache them from there? (Even though the first time the list is displayed, it'll be a bigger set of calls)

from ioctocat.

dennisreimann avatar dennisreimann commented on May 14, 2024

It seems like your email address is not public - at least it's not shown in the API calls:
http://github.com/api/v2/xml/user/show/ussjoin

from ioctocat.

ussjoin avatar ussjoin commented on May 14, 2024

Looks correct-- and most of the people in my feed have similarly non-public email addresses, hence the issue.

It would appear there's not a way to get the picture in the API yet-- which is a bug, but not your bug. So OK, that issue's resolved, but the other issue, of the big black cells with SDK 3.0-- is not.

from ioctocat.

dennisreimann avatar dennisreimann commented on May 14, 2024

Is this fixed in the new version? If not: Can you try to fix this yourself or is there anyone else out there who uses the 3.0 and is able to fix it?

from ioctocat.

ussjoin avatar ussjoin commented on May 14, 2024

This is not fixed under SDK 3.0. I can see what the problem is: if you compile under SDK 3.0 (which, if you're an iPhone developer, is just a matter of going and downloading/installing the new one, then changing the dropdown menu) you have a ton of warnings about deprecated function calls (22 total). Those calls are what's setting everything.

I don't have time this week to attempt to fix this; perhaps next week, but someone else could probably do it faster.

from ioctocat.

dennisreimann avatar dennisreimann commented on May 14, 2024

Relating the black background issue: In the current state the cells are marked as opaqua and are given a white background, can you test whether this works?

from ioctocat.

mccv avatar mccv commented on May 14, 2024

I have some fixes checked in to get rid of deprecation warnings in 3.0. It's basically two instances of

#ifdef __IPHONE_3_0 cell.textLabel.font = [UIFont systemFontOfSize:16.0f]; cell.textLabel.text = repository.name; cell.imageView.image = [UIImage imageNamed:(repository.isPrivate ? @"private.png" : @"public.png")]; #else cell.font = [UIFont systemFontOfSize:16.0f]; cell.text = repository.name; cell.image = [UIImage imageNamed:(repository.isPrivate ? @"private.png" : @"public.png")]; #endif I can't push from my current network location, but can get those in tonight. I think dbloete's fix probably works fine, but probably doesn't hurt to get the deprecations taken care of.

from ioctocat.

dennisreimann avatar dennisreimann commented on May 14, 2024

I'd like to keep this kind of distinguishing out of the code... as long as there are just deprecation warnings we can progress without fixing them and keep this for when the final 3.0 SDK is out...

btw: Good to see that they are making the textLabel and imageView explicit... kinda annoying in the 2.0 SDK as you'll experience weird issues when calling members of own cells like that...

from ioctocat.

mccv avatar mccv commented on May 14, 2024

No worries. I can duplicate the defect on my machine without your latest commits, so I'll try to verify the fix tonight as well.

from ioctocat.

dennisreimann avatar dennisreimann commented on May 14, 2024

Okay, so I finally switched to 3.0 beta 5, because future builds require 3.0 compatibility. Will check the errors this weekend end fix them.

from ioctocat.

intabulas avatar intabulas commented on May 14, 2024

I just ran it under b5 and all the rendering issues described above are no longer present.

from ioctocat.

dennisreimann avatar dennisreimann commented on May 14, 2024

can verify this.

from ioctocat.

ussjoin avatar ussjoin commented on May 14, 2024

I can verify this as well. Sorry for the delay, I was one of those people who got bricked by 3.0b5 and had to do a special downgrade/upgrade maneuver to get the phone back.

from ioctocat.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.