toggleShaderEffects stops functioning after cycling toggleFocusMode #13149

Closed
opened 2026-01-31 03:34:59 +00:00 by claunia · 5 comments
Owner

Originally created by @oising on GitHub (Mar 23, 2021).

Windows Terminal version (or Windows build number)

1.7.572.0

Other Software

No response

Steps to reproduce

  1. install custom shader (e.g. https://github.com/Hammster/windows-terminal-shaders )
  2. bind toggleShaderEffects and toggleFocusMode in settings
  3. toggle shader on
  4. toggle focus on
  5. toggle focus off
  6. toggle shader off
  7. toggle shader on -- this should fail (shader remains off and cannot be turned on). If it turns on, repeat from step 4 - it will fail the second or third time around.

This behavior seems tab-specific, as a new tab will allow the sequence above to be repeated (until that new tab is also unresponsive to toggling shader on/off).

Expected Behavior

shader toggle should always work.

Actual Behavior

shader toggle stops working.

Originally created by @oising on GitHub (Mar 23, 2021). ### Windows Terminal version (or Windows build number) 1.7.572.0 ### Other Software _No response_ ### Steps to reproduce 1) install custom shader (e.g. https://github.com/Hammster/windows-terminal-shaders ) 2) bind toggleShaderEffects and toggleFocusMode in settings 3) toggle shader on 4) toggle focus on 5) toggle focus off 6) toggle shader off 7) toggle shader on -- this should fail (shader remains off and cannot be turned on). If it turns on, repeat from step 4 - it will fail the second or third time around. This behavior seems tab-specific, as a new tab will allow the sequence above to be repeated (until that new tab is also unresponsive to toggling shader on/off). ### Expected Behavior shader toggle should always work. ### Actual Behavior shader toggle stops working.
Author
Owner

@zadjii-msft commented on GitHub (Mar 29, 2021):

Holy crap yea I was able to repro this. What the heck? There's no way that these should interact with each other at all!

FYI I think I hit it by doing 4, 6, 5, 7 - that's the sequence that finally did it for me.

Also for future note:


        { "command": "toggleShaderEffects", "keys": "ctrl+r", "name": "Toggle pixel shader effects" },
        { "command": "toggleFocusMode", "keys": "ctrl+f" },

@zadjii-msft commented on GitHub (Mar 29, 2021): Holy crap yea I was able to repro this. What the heck? There's no way that these should interact with each other _at all_! FYI I think I hit it by doing 4, 6, 5, 7 - that's the sequence that finally did it for me. Also for future note: ```json { "command": "toggleShaderEffects", "keys": "ctrl+r", "name": "Toggle pixel shader effects" }, { "command": "toggleFocusMode", "keys": "ctrl+f" }, ```
Author
Owner

@oising commented on GitHub (Mar 29, 2021):

@zadjii-msft Ok, good -- glad you could repro it. Definitely a heisenbug.

@oising commented on GitHub (Mar 29, 2021): @zadjii-msft Ok, good -- glad you could repro it. Definitely a _heisenbug_.
Author
Owner

@zadjii-msft commented on GitHub (Aug 24, 2022):

Hm. I think this has gone away on newer builds/? At least the old repro case doesn't seem to repro it for me anymore...

@zadjii-msft commented on GitHub (Aug 24, 2022): Hm. I think this has gone away on newer builds/? At least the old repro case doesn't seem to repro it for me anymore...
Author
Owner

@ghost commented on GitHub (Aug 28, 2022):

This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 4 days. It will be closed if no further activity occurs within 3 days of this comment.

@ghost commented on GitHub (Aug 28, 2022): This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for **4 days**. It will be closed if no further activity occurs **within 3 days of this comment**.
Author
Owner

@oising commented on GitHub (Oct 11, 2022):

Hah! A rare occurrence.

@oising commented on GitHub (Oct 11, 2022): Hah! A rare occurrence.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#13149