Skip to content

feature: disable postpone and skip buttons while shortcuts are disabled#731

Open
deltragon wants to merge 3 commits intoslgobinath:masterfrom
deltragon:disable-buttons-initially
Open

feature: disable postpone and skip buttons while shortcuts are disabled#731
deltragon wants to merge 3 commits intoslgobinath:masterfrom
deltragon:disable-buttons-initially

Conversation

@deltragon
Copy link
Collaborator

Description

Resolves #566.

This PR locks the Skip/Postpone button as well while the keyboard shortcuts are disabled.
It also greys out the buttons, and removes the hover style to signal that they are non-interactive.

@deltragon
Copy link
Collaborator Author

I've opened this as a draft because I am unsure about this.
I do think I slightly prefer this over the limitconsecutiveskipping plugin, personally.
However, as this is implemented right now, it will be enabled by default - together with limitconsecutiveskipping. That may be a bit much.
So maybe this should controlled by its own setting, separate from the keyboard shortcuts, and off by default.

@adventuretc
Copy link
Contributor

I always viewed the disabled keyboard shortcuts for n seconds feature as a primarily self-control feature, and only secondarily to prevent accidental keystrokes, so, this PR should definitely be part of the program.

@deltragon deltragon force-pushed the disable-buttons-initially branch from 813aba1 to eabfd22 Compare February 14, 2026 18:43
@deltragon deltragon marked this pull request as ready for review February 14, 2026 18:45
@deltragon
Copy link
Collaborator Author

Well, if others agree, I'm certainly open to merging this. Rebased and fixed the typos.

@archisman-panigrahi
Copy link
Collaborator

Please tell me what I should look for when I test this

@adventuretc
Copy link
Contributor

adventuretc commented Feb 19, 2026

The mouse shouldn't be able to click the skip and postpone button on the break screen until the seconds specified in the second-to-last setting in the Settings.

I personally use this patch and works well on my end.

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.

Add option to disable "skip" and "postpone" buttons for N seconds after break starts

3 participants