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

Add WM_CLASS atom #187

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add WM_CLASS atom #187

wants to merge 3 commits into from

Conversation

tryone144
Copy link

@tryone144 tryone144 commented Jun 9, 2016

As requested here this are the changes to the WM_CLASS atom.

The WM_CLASS is set to "instance", "class", where

element description
class programm name, Bar in this case
instance programm instance, usually the executable name as passed by argv[0]

This should help write better wm rules for hiding the bar and stuff like that.

EDIT: Changed class to capitalized "Bar"

tryone144 added 2 commits June 9, 2016 12:58
Set WM_CLASS class to "bar" and WM_CLASS instance to executable name
(defaults to "lemonbar").
@tryone144 tryone144 mentioned this pull request Jun 9, 2016
Copy link

@ThomasAdam ThomasAdam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tryone144 -- if you're going to set the class name to "bar", give it a capital letter, "Bar". This is the convention to differentiate it absolute from the instance name: "foo, Foo" for example.

Follow convention of capitalized WM_CLASS class name.
@tryone144
Copy link
Author

WM_CLASS class name is now capitalized "Bar" in order to follow the convention to differentiate it from the instance name.

This was referenced Nov 7, 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
Development

Successfully merging this pull request may close these issues.

2 participants