Skip to content
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

Support Tray.setTitle on Linux #4742

Closed
johannesjo opened this issue Mar 10, 2016 · 11 comments
Closed

Support Tray.setTitle on Linux #4742

johannesjo opened this issue Mar 10, 2016 · 11 comments

Comments

@johannesjo
Copy link

  • Electron version: 0.36.10 (latest stable)
  • Operating system: Ubuntu 15.10

I already asked this on stackoverflow: http://stackoverflow.com/questions/35755240/how-to-add-text-to-ubuntu-app-indicator-with-with-nw-js-or-electron

I would like to add text to the ubuntu (unity) app indicator. That's possible via the python package libindicator but not via electron so far.

@zcbenz
Copy link
Member

zcbenz commented Mar 13, 2016

Currently setting label for app indicator is not supported.

@johannesjo
Copy link
Author

What would be required to enable this functionality? Maybe I could provide a PR? I'm just not sure where to start (aside from the libindicator package).

@kevinsawicki kevinsawicki changed the title How to add text to tray on Ubuntu? Support Tray.setTitle on Linux Aug 9, 2016
@johannesjo
Copy link
Author

This is still not supported with electron 1.4.1. I really would like to help, but I'm not sure were to get started.

@johannesjo
Copy link
Author

johannesjo commented Dec 26, 2016

I suppose this would be the right file to map the code: https://github.com/electron/electron/blob/72d67788945d456147edb4498154d231fa1482a6/atom/browser/ui/gtk/app_indicator_icon.cc

Is it enough to bind the function there to the proper app_indicator_set_title?

Also I'm curious about what the best way would be to compile electron and let it run with a proper test case.

@johannesjo
Copy link
Author

johannesjo commented Dec 27, 2016

Ok I found this: https://github.com/electron/electron/blob/master/docs/development/build-instructions-linux.md unfortunately I have no way to make the 25 Gig required available currently on my dev machine. I'm still wondering, if this is not an easy fix.

@MarshallOfSound
Copy link
Member

@johannesjo Try bootstrapping with --dev it will download a lot less

@johannesjo
Copy link
Author

@MarshallOfSound thank you for the tip!

@johannesjo
Copy link
Author

johannesjo commented Dec 28, 2016

Seems like a simple mapping won't solve this issue, as this feature needs to be implemented into chromium itself first (maybe here).

I also wrote a feature request for chromiums issue tracker.

@mkg20001
Copy link

Any progress?

@codebytere
Copy link
Member

Given that this remains impossible without action on the chromium side, and that this isn't currently on our roadmap, i'm going to go ahead and label this a wontfix. However, should someone determine a workable solution or chromium were to implement it, we would be very open to accepting a PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants