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

Fix atom.desktop for some Linux systems #18159

Merged
merged 1 commit into from Sep 29, 2018

Conversation

Projects
None yet
2 participants
@shadow-light
Contributor

shadow-light commented Sep 29, 2018

For some linux systems the window manager does not associate atom.desktop with Atom processes unless the class name is in lowercase. See #16791

Fix atom.desktop for some Linux systems
For some linux systems the window manager does not associate atom.desktop with Atom processes unless the class name is in lowercase. See #16791
@daviwil

This comment has been minimized.

Member

daviwil commented Sep 29, 2018

Thanks @shadow-light! This change looks fine to me. Out of curiosity, which distro and desktop environment are you using?

@daviwil daviwil merged commit eda0c74 into atom:master Sep 29, 2018

3 checks passed

Atom Pull Requests #20180929.1 succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@shadow-light

This comment has been minimized.

Contributor

shadow-light commented Sep 30, 2018

👍 I'm using Ubuntu 16.04 which uses Unity.

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