Skip to content

Fix Super+Arrow bindings: split xfconf-query reset and create#23

Merged
kimptoc merged 1 commit intokimptoc:claude/fix-xfce-wallpaper-FK5rhfrom
cattyclaw-bot:claude/fix-xfce-wallpaper-FK5rh
May 4, 2026
Merged

Fix Super+Arrow bindings: split xfconf-query reset and create#23
kimptoc merged 1 commit intokimptoc:claude/fix-xfce-wallpaper-FK5rhfrom
cattyclaw-bot:claude/fix-xfce-wallpaper-FK5rh

Conversation

@cattyclaw-bot
Copy link
Copy Markdown
Contributor

Follow-up to #22.

The first test boot from the merged xfce branch produced 8× this in ~/myautostart.log:

--create and --reset options can not be used together.

xfconf-query 4.20 (the version in our nixpkgs closure) refuses -r and -n in one invocation, so the combined-flag form #22 settled on doesn't actually work — every Super+Arrow write silently failed, and the trailing xfwm4 Super+Arrow tile keys bound line in the log is just an unconditional echo, not a status check.

Split each binding into two separate xfconf-query calls:

$XQ -p '/xfwm4/default/<Super>Left' -r 2>/dev/null || true
$XQ -p '/xfwm4/default/<Super>Left' -n -t string -s 'tile_left_key' 2>>$MYLOG

This is the same pattern fd9e958 already used for Super+Up — now applied to all 8 bindings.

Test plan

  • Build the ISO from this branch and boot it
  • grep -E 'create.*reset|Super|tile' ~/myautostart.log — expect no "options can not be used together" lines
  • xfconf-query -c xfce4-keyboard-shortcuts -l -v | grep '/xfwm4/default/<Super>' — expect all 8 bindings listed
  • Manual: focus a window, test Super+Left/Right/Up/Down and the corner combos Super+Alt+Arrow, Super+Ctrl+Arrow

🤖 Generated with Claude Code

…alls

xfconf-query 4.20 (the version in the closure) refuses -r and -n in the
same invocation: "--create and --reset options can not be used together".
The previous attempt to fold reset+create into one call produced 8 such
errors per boot in ~/myautostart.log (one per binding) and left every
binding unset — the trailing "Super+Arrow tile keys bound" line in the
log is just an unconditional echo, not a status check.

Issue each -r and -n as separate calls (the original Super+Up pattern),
extended to all 8 bindings.
@kimptoc kimptoc merged commit 3b67111 into kimptoc:claude/fix-xfce-wallpaper-FK5rh May 4, 2026
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