ionic-team / ionic-v1

The repo for Ionic 1.x. For the latest version of Ionic, please see https://github.com/ionic-team/ionic
Other
193 stars 187 forks source link

overflow-scroll can not set dynamically #231

Closed brandyscarney closed 5 years ago

brandyscarney commented 7 years ago

From @johnnynode on January 10, 2017 5:31

Ionic version: (check one with "x") [x] 1.x [ ] 2.x

I'm submitting a ... (check one with "x") [ ] bug report [x] feature request [ ] support request

Current behavior:
There it is :

<ion-content overflow-scroll="the issue is here,can't set synamically only string type supported" scroll="same issue here only true and false, variable not supported"></ion-content>

can not use expression to set in different platforms or situations. And, can not use ng-attr-overflow-scroll="{{expression here}}" , not working as well.

Copied from original issue: driftyco/ionic#9930

VinceOPS commented 7 years ago

Would be (very) "nice to have", +1.

I made a commit to my own repository of ionic (v1) to handle basic ng expressions: https://github.com/VinceOPS/ionic/commit/5c35c6a9ba971dddc5e4b966e336b4ff0f782a66

Be careful if you use this fork, there are quite a few changes/cutomizations.

NB: I did not copied/pasted the changes for ion-scroll, but we could, too. I just don't need it...