Skip to content

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

Merged
archisman-panigrahi merged 3 commits into
slgobinath:masterfrom
deltragon:disable-buttons-initially
Mar 29, 2026
Merged

archisman-panigrahi merged 3 commits into
slgobinath:masterfrom
deltragon:disable-buttons-initially

Conversation

@deltragon

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Collaborator Author

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

@archisman-panigrahi

Copy link
Copy Markdown
Collaborator

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

@adventuretc

adventuretc commented Feb 19, 2026 •

Copy link
Copy Markdown
Contributor

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.

@archisman-panigrahi

Copy link
Copy Markdown
Collaborator

I tested that it works. Very sorry about the delayed response.

@archisman-panigrahi
archisman-panigrahi merged commit 389347e into slgobinath:master Mar 29, 2026
6 checks passed
@deltragon
deltragon deleted the disable-buttons-initially branch April 12, 2026 17:17
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