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

X.A.UpKeys: Init #900

Merged
merged 1 commit into from
Aug 27, 2024
Merged

X.A.UpKeys: Init #900

merged 1 commit into from
Aug 27, 2024

Conversation

slotThe
Copy link
Member

@slotThe slotThe commented Aug 22, 2024

Description

I've known about this stackoverflow post for a while, but seemingly the code has never made it to contrib. I think that it could come in handy once in a while, so why not add it?

Checklist

  • I've read CONTRIBUTING.md

  • I've considered how to best test these changes (property, unit,
    manually, ...) and concluded: I tested the extended example and things seem to work nicely; no actual tests (yet‽)

  • I updated the CHANGES.md file

    TODO

Next, define the keys and actions you want to have happen on the release
of a key:

> myUpKeys = ezUpKeys $
Copy link
Member Author

Choose a reason for hiding this comment

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

There are, I guess, a million ways one could go about implementing this. Do we ask users to specify whether they want the key grabbed for every key (possibly with a global default fallback)? Do we also accept bindings for key presses here, so that one doesn't need to "split" the functionality across different parts of the config? I haven't thought too much about this, and just went with the most obvious one. Open for opinions

@slotThe slotThe merged commit d0d9d42 into xmonad:master Aug 27, 2024
20 checks passed
@slotThe slotThe deleted the x.a.upkeys branch August 27, 2024 06:26
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