setchi / FancyScrollView

[Unity] Scroll view component that can implement highly flexible animations.
https://setchi.jp/FancyScrollView/
MIT License
3.04k stars 385 forks source link

スクロールできない場合padding headが効かない #49

Closed square1983 closed 4 years ago

square1983 commented 4 years ago

お疲れ様です、エンジニアの楊と申します。 今担当している案件ではFancyScrollViewライブラリは使われています、とても優秀なライブラリと思いました、ありがとうございました。 開発中で一点気になったことがあります。スクロールビューの中セル数が多い場合(スクロールバーが表示される)padding headは効いてますけど、セル数が少ない場合(スクロールバーが表示されない)padding head設定は効かない、一番目のセルとTopの距離は固定されています。 ソースコードを拝見しまして、FancyScrollRect.cs#L108のところ、base.UpdatePosition(Scrollable ? ToFancyScrollViewPosition(p) : 0f);処理は分岐されています。ここは何か意図はございますか?スクロールできるできないににかかわらず、同じpadding headにしたい場合は如何設定すれば良いでしょうか? お手数ですが、ご確認のほど宜しくお願い致します。

setchi commented 4 years ago

@square1983 ご報告ありがとうございます。

実装が間違っていました。下記が本来の意図だったので修正させていただきました。

base.UpdatePosition(ToFancyScrollViewPosition(Scrollable ? p : 0f));

よろしくお願いいたします。

alim-youken commented 4 years ago

@setchi ご対応ありがとうございました、大変助かりました。

square1983 commented 4 years ago

@setchi ご対応ありがとうございました。