search

ebryn / ember-titanium

Titanium Mobile wrapped in Ember (formerly SproutCore-Titanium)
41 stars 2 forks source link

Need for a less inclusive base class than SCTi.View #4

Closed brianpattison closed 13 years ago

brianpattison commented 13 years ago

SCTi.AlertDialog extending SCTi.View doesn't make sense. We need a base class that doesn't have any properties or child views.

We should also be using SproutCore Mixins. Have a mixin called SCTi.Openable rather than TabGroup extending the Window class. TabGroup doesn't really share any properties, methods, or events with Window other than open() and close().

ebryn commented 13 years ago

Sounds good, like your approach in the pull request.