Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TIMOB-6398 prop() #865

Merged
merged 40 commits into from Dec 14, 2011
Merged

TIMOB-6398 prop() #865

merged 40 commits into from Dec 14, 2011

Conversation

cb1kenobi
Copy link
Contributor

Implements an internal-only function prop() that defines properties of the API classes.

get: function(){return obj.dom.style.backgroundPositionX;},
set: function(val){obj.dom.style.backgroundPositionX = val + "px"}
set: function(val){return obj.dom.style.backgroundPositionX = val + "px"}
});
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is more of a note for when redo the UI architecture, but this setter would break if val is something like "50px"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed!

@nebrius
Copy link
Contributor

nebrius commented Dec 8, 2011

Code reviewed. Request accepted

@Mindelusions
Copy link
Contributor

Reviewed and accepted

@donthorp
Copy link
Contributor

Reviewed and accepted. @cb1kenobi and @Mindelusions tested this extensively.

donthorp added a commit that referenced this pull request Dec 14, 2011
@donthorp donthorp merged commit cc5f4fc into tidev:master Dec 14, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants