fix issue #41

[ashconnell]

This fixed #41 and adds a foreground boolean the same as on Android

[macdonst]

@Deminetix can you modify your fix so that is sets foreground to true when isInline is true and sets foreground to false when isInline is false. Then I'll test your PR and merge.

[ashconnell]

@macdonst i'm not sure i follow. What was the original intention of the isInline variable?

[macdonst]

I don't even think we need to be this fancy. The 'notificationReceived' method only gets called when the app is the the foreground.

[macdonst]

@Deminetix okay, I merged this in but I will do the quick addition to set foreground to false myself.

I really appreciate you stepping up to help out with this plugin. Finally a minor point, check out Chris' slides on commit messages. Something like this PR could be rebased so it is all one commit instead of 3.

https://github.com/CSilivestru/git-different

[lock]

This thread has been automatically locked.