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

Update ODSAccordionView.m #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wedkarz
Copy link

@wedkarz wedkarz commented Feb 16, 2015

Fix for scroll content size, when frame.top isn't 0.

Fix for scroll content size, when frame.top isn't 0.
@johanneswseitz
Copy link
Owner

Why do you think this fix is necessary? The way I see it self.frame.origin.y is the position of the accordion view relative to the parent view. How does the position affect the size of the accordion content in any way?

@wedkarz
Copy link
Author

wedkarz commented Feb 22, 2015

When you embed accordion in other view so that it's top coordinate isn't 0,
current size doesn't take it into account. Effectively you see accordion in
some frame and it's impossible to scroll to very bottom.

2015-02-16 23:06 GMT+01:00 Johannes Seitz [email protected]:

Why do you think this fix is necessary? The way I see it
self.frame.origin.y is the position of the accordion view relative to
the parent view. How does the position affect the size of the
accordion content in any way?


Reply to this email directly or view it on GitHub
#5 (comment)
.

Pozdrawiam,
Artur Rybak

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