Skip to content

grimblast: support cursor in frozen area capture - #196

Merged
fufexan merged 3 commits into
hyprwm:mainfrom
shovel-kun:grimblast-cursor-freeze-area
Oct 5, 2026
Merged

fufexan merged 3 commits into
hyprwm:mainfrom
shovel-kun:grimblast-cursor-freeze-area

Conversation

@shovel-kun

Copy link
Copy Markdown
Contributor

Description of changes

Allow grimblast --freeze --cursor ... area now that hyprpicker can include the cursor in its frozen screencopy.

Fixes #92 (with the caveat that it only works with hardware cursors, software cursors still borked).
Depends on hyprwm/hyprpicker#156.

  • For new programs

    • Add the program to the README table, with yourself as the maintainer
    • Add a README for the program itself
    • Add Makefile (and Nix derivation optionally, otherwise @fufexan will do it)
    • If the program is a script, add it to the CI checks matrix
  • For changes

    • Add changes to the CHANGELOG
    • Reflect your changes in the man pages (if they exist)

Example: (notice there's no double cursor)

image

@fufexan fufexan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why have CURSOR and CAPTURE_CURSOR? Can't we use one variable?

Also, how will this work with existing hyprpicker versions?

@shovel-kun

Copy link
Copy Markdown
Contributor Author

@fufexan Sry for the late reply

Why have CURSOR and CAPTURE_CURSOR? Can't we use one variable?

Woops, we could use CURSOR for both yeah.

Also, how will this work with existing hyprpicker versions?

Maybe I should do a check for the --cursor option existing in hyprpicker --help? Not too familiar with shell scripts.

@fufexan

fufexan commented May 8, 2026

Copy link
Copy Markdown
Member

Maybe I should do a check for the --cursor option existing in hyprpicker --help? Not too familiar with shell scripts.

Sure, can be done.

@shovel-kun

Copy link
Copy Markdown
Contributor Author

@fufexan alrighty

@shovel-kun

Copy link
Copy Markdown
Contributor Author

@fufexan hi, wondering if we could get this merged

@fufexan

fufexan commented Oct 1, 2026

Copy link
Copy Markdown
Member

Oops, I did not see your comment in may. We can, but it needs another rebase.

@shovel-kun
shovel-kun force-pushed the grimblast-cursor-freeze-area branch from 19baa12 to 8cf5e24 Compare October 5, 2026 16:56
@shovel-kun

Copy link
Copy Markdown
Contributor Author

@fufexan That's alright! Rebased and also fixed clicking a window with --freeze --cursor causes dropping of the cursor

@fufexan

fufexan commented Oct 5, 2026

Copy link
Copy Markdown
Member

My bad, I merged something and the changelog now conflicts.

@fufexan
fufexan force-pushed the grimblast-cursor-freeze-area branch from 33612de to d8613eb Compare October 5, 2026 18:59
@fufexan
fufexan merged commit a2b16bd into hyprwm:main Oct 5, 2026
@shovel-kun

Copy link
Copy Markdown
Contributor Author

Thanks!

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.

--cursor flag doesn't work in grimblast

2 participants