shima11 / FlexiblePageControl

A flexible UIPageControl like Instagram.
MIT License
797 stars 108 forks source link

fixes ScrollViews layout. #27

Closed TomoyaOnishi closed 5 years ago

TomoyaOnishi commented 5 years ago

はじめまして!

レイアウトが完了してから再度numberOfPagesを変更するような処理をすると、Viewの位置がズレるようです。こちらを修正してみました。いかがでしょうか?

Demoで以下のようなコードを追加すると再現できます。

    override func viewDidAppear(_ animated: Bool) {
        super.viewDidAppear(animated)
        pageControl1.numberOfPages = 10
        pageControl2.numberOfPages = 10
    }

scrollView.frameを変更せず、bounds.sizeを変更することで位置を変えずに大きさだけ変更するようにしました。

ページ数を都度変えるような実装もあるかと思いますので、ご確認よろしくお願いいたします 🙇

shima11 commented 5 years ago

PRありがとうございます! 遅くなってしまいすいません 🙇 確認してみます。

shima11 commented 5 years ago

いまのところ問題なさそうですね 一旦、マージしちゃいます!

TomoyaOnishi commented 5 years ago

ありがとうございます😊