Breaks UITextView !!! about ckuitools HOT 13 CLOSED

I think this is fixed - will confirm ASAP.

from ckuitools.

i think commit c6322c1 fixed it

d.

On Oct 4, 2013, at 8:02, Jasper Blues [email protected] wrote:

I think this is fixed - will confirm ASAP.

—
Reply to this email directly or view it on GitHub.

from ckuitools.

I've been following this issue for while and I wonder how could you ever reproduce it:

1. @genericspecific, how can I spot the invocation of -[UITextView setSize:] method (the commit above mentions it explicitly). In which circumstances is it called by UITextView?
2. @jasperblues, I tried my best to reproduce this both programmatically and using storyboards (I did it on 0.5.0 - a half year ago release). How did you ever do that?

I am wondering because I can't put up with the change c6322c1 commit introduces: UIView's property @SiZe to @viewSize renaming and want to see the real case where the need in such renaming becomes obvious.

Thanks.

from ckuitools.

Steps to reproduce:

1. Install CKUITools via CocoaPods
2. Create a project
3. Programmatically add a UITextView to a UITableViewCell

Run. Enjoy.

from ckuitools.

http://stackoverflow.com/questions/16118106/uitextview-strange-text-clipping

from ckuitools.

OK, thanks for the fast response!

from ckuitools.

try this:

UITextView *tv = [[UITextView alloc] init];
NSLog(@"set size exists = %i" , [tv respondsToSelector:@selector(setSize:)]);

d.

On Oct 4, 2013, at 11:50, Stanislaw Pankevich [email protected] wrote:

I've been following this issue for while and I wonder how could you ever reproduce it:

@genericspecific, how can I spot the invocation of -[UITextView setSize:] method (the commit above mentions it explicitly). In which circumstances is it called by UITextView?
@jasperblues, I tried my best to reproduce this both programmatically and using storyboards (I did it on 0.5.0 - a half year ago release). How did you ever do that?
I am wondering because I can't put up with the change c6322c1 commit introduces: UIView's property @SiZe to @viewSize renaming and want to see the real case where the need in such renaming becomes obvious.

Thanks.

—
Reply to this email directly or view it on GitHub.

from ckuitools.

No worries Stanislaw :)

from ckuitools.

Yes, it is.

2013-10-04 12:57:30.891 SandboxApp[9369:a0b] set size exists = 1

I want to think about it and come up with a decent consistent solution: having x, y, origin, ... and ... viewSize breaks my heart ))

Again, thanks for clarification!

from ckuitools.

swizzling! ;)

from ckuitools.

This is the follow-up:

SEL selectors[8] = {
    @selector(setSize:),
    @selector(setOrigin:),
    @selector(setX:),
    @selector(setY:),
    @selector(setWidth:),
    @selector(setHeight:),
    @selector(setCenterX:),
    @selector(setCenterY:)
};

for (int i = 0; i < 8; i++) {
    NSLog(@"%@: %d", NSStringFromSelector(selectors[i]), [[UITextView new] respondsToSelector:selectors[i]]);
}

Looks like origin is also a bad guy! Thoughts?

from ckuitools.

Yup. I use position rather than origin.

On Oct 4, 2013, at 12:25, Stanislaw Pankevich [email protected] wrote:

This is the follow-up:

SEL selectors[8] = {
@selector(setSize:),
@selector(setOrigin:),
@selector(setX:),
@selector(setY:),
@selector(setWidth:),
@selector(setHeight:),
@selector(setCenterX:),
@selector(setCenterY:)
};

for (int i = 0; i < 8; i++) {
NSLog(@"%@: %d", NSStringFromSelector(selectors[i]), [[UITextView new] respondsToSelector:selectors[i]]);
}
Looks like origin is also a bad guy! Thoughts?

—
Reply to this email directly or view it on GitHub.

from ckuitools.

Ah, I see - position is here in CKUITools.

I think, I will choose more conservative option: @viewSize and @viewOrigin. Two of them is a band 👍 !

from ckuitools.