Skip to content

allow enable/disable undo/redo buttons#22

Merged
eagleoflqj merged 3 commits intofcitx-contrib:masterfrom
WhiredPlanck:button-enable
Mar 30, 2025
Merged

allow enable/disable undo/redo buttons#22
eagleoflqj merged 3 commits intofcitx-contrib:masterfrom
WhiredPlanck:button-enable

Conversation

@WhiredPlanck
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Member

@eagleoflqj eagleoflqj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When button is disabled, background color should not change on touch.

@WhiredPlanck
Copy link
Copy Markdown
Contributor Author

When button is disabled, background color should not change on touch.

Changed. It may not be the best practice, feel free to polish.

Comment thread src/toolbar.ts Outdated
Copy link
Copy Markdown
Member

@eagleoflqj eagleoflqj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I revert default to enabled. I think on Android you don't need to use this feature since you don't manage stacks. Also on Harmony we have to disable them on input context switch anyway.

@eagleoflqj eagleoflqj merged commit 249fae8 into fcitx-contrib:master Mar 30, 2025
1 check passed
@WhiredPlanck WhiredPlanck deleted the button-enable branch March 30, 2025 18:21
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