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

Use App indicator when running under Enlightenment #1941

Merged
merged 1 commit into from May 13, 2016
Merged

Use App indicator when running under Enlightenment #1941

merged 1 commit into from May 13, 2016

Conversation

@J5lx
Copy link
Contributor

@J5lx J5lx commented May 13, 2016

Some time ago, Enlightenment dropped its XEmbed support since it had never functioned well anyway (See e.g. https://twitter.com/_Enlightenment_/status/538000507315314688). Therefore, an app indicator has to be used when running under Enlightenment as well.

I'm not a Python programmer, so I'm not quite sure whether the way I indented the long lines is "correct" and I couldn't find good examples in QL's source tree within a reasonable amount of time.

@declension
Copy link
Member

@declension declension commented May 13, 2016

LGTM

@declension declension merged commit ef4c14e into quodlibet:master May 13, 2016
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@declension
Copy link
Member

@declension declension commented May 13, 2016

Thanks @J5lx

@J5lx J5lx deleted the J5lx:enlightenment-appindicator branch May 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants