Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add CTRL+A shortcut #4041

Merged
merged 2 commits into from
Jan 11, 2021
Merged

Add CTRL+A shortcut #4041

merged 2 commits into from
Jan 11, 2021

Conversation

winkies
Copy link
Contributor

@winkies winkies commented Sep 29, 2020

Fixes #2155

Description of the Change
Define shortcut on accelerator table and bind it with event table.

Alternate Designs
Tested only on Linux environment. It should work as well on Windows but it should be tested on MacOS X.

Release Notes
Add CTRL+A shortcut for advance view of boinc manager.

@winkies winkies changed the title Add shortcut CTRL+A Add CTRL+A shortcut Sep 29, 2020
@RichardHaselgrove
Copy link
Contributor

Tested on Windows: it does work on the Tasks tab, which is where it will be most useful for me (see #3778).

I rarely need to operate on all tasks across all projects, but I might need to operate on all tasks from a single project. It would be helpful to have a command button "Show [tasks from] only this project", analogous to the button at the foot of the Event Log window - the two controls would work well together.

@AenBleidd
Copy link
Member

@RichardHaselgrove, sound like a new feature request ;)

@winkies
Copy link
Contributor Author

winkies commented Sep 29, 2020

@RichardHaselgrove Glad it's useful :)

I see what you need : it doesn't seem to be too much difficult.
If maintainer (@AenBleidd ) approves your feature request, I could implement it in another one pull request ;)

@RichardHaselgrove
Copy link
Contributor

The test for me of how useful a feature is: how many machines do I install in on, on the first day of testing.

Three hosts so far, monitoring eight clients between them!

So, yes, I propose "Show [tasks from] only this project" as a feature request, and I look forward to testing the resulting PR!

@RichardHaselgrove
Copy link
Contributor

@winkies: not wanting to overwhelm you, but if you're comfortable working with the keyboard handler in wxWidgets, may I draw your attention to my #3359 - that one might be up your street too.

@AenBleidd AenBleidd merged commit 9caf3df into BOINC:master Jan 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CTRL+A shortcut doesn't work
3 participants