Minutes of the core team meeting-2014/03/14


-Passing

Trek to Glovac

Although most of our collaboration is conducted on Github, IRC (#emberjs On freenode.net), and our Discourse site
This Ember.js core team Meet in private via Google Hangout every Friday at 2pm Eastern Standard Time/11am Pacific Standard Time to discuss everything about Ember every week.

If you have a topic you want to discuss, please contact your favorite core team member and let them know!

participant

@ebryn, @krisselden, @machty, @rwjblue, @stefanpenner, @tomdale, @trek, @wycats

topic

Pass/Fail function list

  • query-params-new #4008
    Still not feasible, but @machty and @wycats elaborate on the details

PR/issues to review

  • canSetInnerHTML: IE cannot set innerHTML on multiple tags

    IE does not support .innerHTML = on COL, COLGROUP, FRAMESET, HTML, STYLE, TABLE, TBODY, TFOOT, THEAD, TITLE, or TR.

    Solution: It is definitely a bug, but this will be fixed by HTMLbars, so we would rather wait for it to log in than fix it now

  • Override the observer and listener in Ember.CoreObject.create().

    Modify the mixin to include the code makeCtor Remove any observers/listeners set on the attribute or key before adding the attribute.

    What I care about is the modification in this particular code path.

    Solution: This used to be how create worked, but for performance reasons, we gave up on this. .extend
    For use at design time, not at run time: use .extend().create() or .createWithMixins()

  • Use injected test helpers instead of local functions.

    Since we are using our helper (into the App.testHelpers hash) to inject the application, we should use that helper instead of calling the function in the local scope.

    As the test shows, this allows users to override the internal assistant with a version tailored to their scenario.

    Solution: Make it possible, but still a private API. Assert when trying to override the built-in helper.


Please enable JavaScript to view

Comments are powered by Disqus.


Leave a Reply

Your email address will not be published. Required fields are marked *