Skip to content

grimblast: add foreign toplevel support for window selection#199

Open
zspher wants to merge 1 commit into
hyprwm:mainfrom
zspher:feature-grimblast-toplevel-handle
Open

grimblast: add foreign toplevel support for window selection#199
zspher wants to merge 1 commit into
hyprwm:mainfrom
zspher:feature-grimblast-toplevel-handle

Conversation

@zspher
Copy link
Copy Markdown

@zspher zspher commented May 18, 2026

Description of changes

Adds foreign toplevel support for window selection.
closes #198

behavior changes:

  • allows taking screenshots of windows behind floating windows
  • allows taking screenshots of selected windows in another workspace (i.e. you switched workspace while -w)

Things done

  • 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)

@zspher zspher changed the title grimblast: add toplevel handle support for window selection grimblast: add foreign toplevel support for window selection May 18, 2026
@zspher zspher force-pushed the feature-grimblast-toplevel-handle branch from e6871d6 to d8f8780 Compare May 18, 2026 02:57
@zspher zspher marked this pull request as draft May 18, 2026 03:06
@zspher zspher force-pushed the feature-grimblast-toplevel-handle branch 2 times, most recently from ebfb628 to 882a0c5 Compare May 18, 2026 04:09
@zspher
Copy link
Copy Markdown
Author

zspher commented May 18, 2026

sry, for the spam, I'm still learning bash. I was fixing the custom area selection.

anyway I think this should be ready for review

@zspher zspher marked this pull request as ready for review May 18, 2026 04:30
@zspher zspher marked this pull request as draft May 18, 2026 05:43
@zspher zspher force-pushed the feature-grimblast-toplevel-handle branch from 882a0c5 to 1ec8201 Compare May 18, 2026 06:27
@fufexan
Copy link
Copy Markdown
Member

fufexan commented May 18, 2026

Do I need a specific version of grim? I have 1.5.0. I've tried this and grim is killed

› grimblast/grimblast --notify copy area
grim -t png -T 1800000f -
xargs: grim: terminated by signal 11

@zspher
Copy link
Copy Markdown
Author

zspher commented May 18, 2026

ok, thats wierd. I'm also using grim 1.5.0 w/ hyprland 0.55.1.

can you check if grim -T $(hyprctl -j activewindow | jq -r .stableId) works?
also can you pull the latest branch (sry, for the multiple force-push)?

@zspher zspher marked this pull request as ready for review May 20, 2026 20:19
@fufexan
Copy link
Copy Markdown
Member

fufexan commented May 22, 2026

Sorry for the late reply

› grim -T $(hyprctl -j activewindow | jq -r .stableId)
zsh: segmentation fault (core dumped)  grim -T $(hyprctl -j activewindow | jq -r .stableId)

@zspher
Copy link
Copy Markdown
Author

zspher commented May 22, 2026

hi,
best I can tell is this might be an issue with hyprland screenshare and I don't know how to fix that
tho you may want to look at hyprwm/Hyprland#14123 for how to diagnose

@kRHYME7
Copy link
Copy Markdown

kRHYME7 commented Jun 3, 2026

works fine on my end. been using it with window switcher preview. using hyprland-git and latest grim from arch.

image trying to stretch it out of bounds. Also integrated it on my dots. seems non is broken.

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.

feature: use foreign toplevel -T flag for window capture in grim for grimblast

3 participants