Skip to content

Add livescript support and diable growl notifications #50

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

Closed
wants to merge 1 commit into from
Closed

Add livescript support and diable growl notifications #50

wants to merge 1 commit into from

Conversation

tiye
Copy link
Contributor

@tiye tiye commented Oct 22, 2012

  • add livescript support(use nearly the same code as the code for coffeescript)
  • disable the notifications of growl.. not a good Idea, though.

The problem I came across is that there are always too many icons stay in GNOME3, and here's my screenshot.
I tested growl({sticky: false}) before but didn't work as expetcted. So I'm trying to diable it.
But that's also a bad solution...

http://forum.ubuntu.org.cn/viewtopic.php?f=48&t=385998
too many

@fgnass
Copy link
Owner

fgnass commented Oct 22, 2012

I added --hint=int:transient:1 to the notify-send call so that the notifications should no longer pile up:
fgnass/node-growl@f7d592c

Could try node-dev with my node-growl fork on your machine? If this indeed fixes the issue I'll send TJ a pull-request.

@tiye
Copy link
Contributor Author

tiye commented Oct 22, 2012

Thanks. It don't pile up now.

To be honest, I don't think it's that useful to pop up the notifications.
Personaly I just keep my terminal in the corner on the screen.
The notifications poped is not as useful as the restarting info in the terminal.
So I think it's not the best idea to use poped up notifications.
fullscr

@tiye
Copy link
Contributor Author

tiye commented Nov 2, 2012

@fgnass removed the commit about growl.. now only to add support for LiveScript.

This was referenced Nov 7, 2012
@tiye tiye closed this Dec 5, 2012
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.

2 participants