Added checkbox to invert mouse wheel direction in window's titlebar and frame actions. #220

Merged
MicheleC merged 1 commits from feat/inverse-wheel into master 3 years ago
Owner

This resolves issue TDE/tde#64.

Attached what it looks like. Let me know if you think we should place the checkbox in different places.

This resolves issue TDE/tde#64. Attached what it looks like. Let me know if you think we should place the checkbox in different places.
MicheleC added the PR/rfc label 3 years ago
MicheleC added this to the R14.0.11 release milestone 3 years ago
MicheleC force-pushed feat/inverse-wheel from c0c12796cf to 5dc80ceb58 3 years ago

This looks great! Clear and useful.

I suggest, however, that the phrase should be "Reverse mouse direction", as "reverse" feels like it applies to "direction" more naturally than "invert" does.

This looks great! Clear and useful. I suggest, however, that the phrase should be "Reverse mouse direction", as "reverse" feels like it applies to "direction" more naturally than "invert" does.
Poster
Owner

Yes, I was undecided between "reverse" and "invert" actually.
I have just looked at the Peripherals --> Mouse module and there "reverse" is used, so I will use the same word for consistency.

Yes, I was undecided between "reverse" and "invert" actually. I have just looked at the Peripherals --> Mouse module and there "reverse" is used, so I will use the same word for consistency.
Owner

The "reverse" sounds good. The GUI design looks great. Good work!

The "reverse" sounds good. The GUI design looks great. Good work!
MicheleC force-pushed feat/inverse-wheel from 5dc80ceb58 to 3e9ff8ce3b 3 years ago
MicheleC force-pushed feat/inverse-wheel from 3e9ff8ce3b to 3da4fc88bf 3 years ago
MicheleC force-pushed feat/inverse-wheel from 3da4fc88bf to 11a496c359 3 years ago
MicheleC merged commit 11a496c359 into master 3 years ago
MicheleC removed the PR/rfc label 3 years ago
MicheleC deleted branch feat/inverse-wheel 3 years ago
Poster
Owner

Thanks for the feedback guys. Merged and backported to R14.0.11 too.

Thanks for the feedback guys. Merged and backported to R14.0.11 too.
The pull request has been merged as 11a496c359.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: TDE/tdebase#220
Loading…
There is no content yet.