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 details to ManageHook.hs docs #504
Conversation
src/XMonad/ManageHook.hs
Outdated
@@ -91,7 +92,7 @@ className :: Query String | |||
className = ask >>= (\w -> liftX $ withDisplay $ \d -> fmap resClass $ io $ getClassHint d w) | |||
|
|||
-- | A query that can return an arbitrary X property of type 'String', | |||
-- identified by name. | |||
-- identified by name. Works for ASCII strings only. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you perhaps wish to add a note that for _NET_WM_NAME
, WM_NAME
and possibly also WM_CLASS
there are special functions available that work better, and hyperlink to them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, quite right.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’ve just added a short sentence referring to the specialized variants. Is that fine? Any more suggestions?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, I think this is perfect!
(just a typo there, then lgtm)
Thanks! |
Description
Add relevant information about
title
andstringProperty
as discussed in #503.Checklist
I've read CONTRIBUTING.md
I've confirmed these changes don't belong in xmonad-contrib instead
I've considered how to best test these changes (property, unit,
manually, ...) and concluded: XXX
I updated the
CHANGES.md
fileWell, if you think, I shall test these doc changes or update the
CHANGES.md
, please tell me.