Wei Wang

Results 57 comments of Wei Wang

New style , pretty, but simple

Thanks. I will review tomorrow.

Add context menus

This is interesting. I will try to find time to code review this, but I don't find that I have a lot of time these days to do so. Please...

Improvements to the style & to the text content

@JakeGinnivan I'm not sure I'm ready to create an org just yet. I have some work in mind that I wanted to do in this project before I open it...

Make the specific examples linkable

This is a good suggestion. I will explore some options.

When a tag is checked out, current branch should say "DETACHED HEAD"

@JakeGinnivan

[api] Add removeListenersByContext(context) method

Hmm, not really sure what this username and accessKey stuff for SAUCE is all about. Unit tests pass for sure.

[api] Add removeListenersByContext(context) method

@lpinca Thanks. Method renamed.

[api] Add removeListenersByContext(context) method

When using EventEmitter with any sort of browser-side framework such as Angular or React, a common pattern is such: ```jsx export default class MyReactComponent { componentDidMount() { // someEE is...

[api] Add removeListenersByContext(context) method

@3rd-Eden Using fat arrow functions only solves the `addEventListener` part, but it does not solve the `removeListener` part of a component's lifecycle since the lifecycle functions are different methods.

isparta instrumenter is not working with karma-coverage v0.5.0

@martinmicunda I think you mean `v0.5.0`. I also just tried with `v0.5.2` and I am still seeing ES5 and not ES6 code.