Skip to content

Fix and consolidate the way that alignment works #784

Open
publickeating opened this Issue May 19, 2012 · 1 comment

2 participants

@publickeating
SproutCore member

For example, SC.ScrollView uses two properties, horizontalAlign and verticalAlign, while SC.ImageViews uses a single property align:

We should only have one version used for all views.

Edit: It doesn't appear that SC.ScrollView alignment works at all, so it should be fixed too.

@dcporter
SproutCore member

SC.ScrollView alignment doesn't work because ScrollView (unexpectedly) does its positioning with scrollTop, which can't be less than zero (req. for centering or bottoming content) on desktop browsers. Why scrollTop?!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.