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

Use GAction where possible #1927

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Conversation

infirit
Copy link
Contributor

@infirit infirit commented Nov 12, 2022

Eventually we will need to drop the current menu's for something else. In preparation I wanted to move all "regular" menuitem callbacks to GAction as pretty much all replacements will use them . Sadly RadioMenuItems don't work with GAction 😞.

I think it would be good to do as much as possible so marked it for review. Also included accelerators for closing windows and apps.

@infirit infirit changed the title blueman-manager" Experiment with GAction blueman-manager Experiment with GAction Nov 21, 2022
@infirit infirit changed the title blueman-manager Experiment with GAction blueman-manager: Experiment with GAction Nov 21, 2022
@infirit infirit force-pushed the gaction branch 2 times, most recently from d34f7be to 84b1c81 Compare December 10, 2022 17:02
@infirit infirit changed the title blueman-manager: Experiment with GAction Use GAction where possible and add accelerators Dec 10, 2022
@infirit infirit marked this pull request as ready for review December 10, 2022 17:56
@infirit infirit linked an issue Dec 10, 2022 that may be closed by this pull request
1 task
@infirit infirit requested a review from cschramm December 12, 2022 13:56
@sonarqubecloud

This comment was marked as outdated.

@infirit
Copy link
Contributor Author

infirit commented Dec 20, 2022

I have reworked this a bit and added wip ManagerDeviceList actions. I am thinking now how to handle plugins...

I'll split off the keyboard shortcuts into a separate PR soon.

@infirit infirit removed the request for review from cschramm December 20, 2022 20:06
@infirit infirit marked this pull request as draft January 2, 2023 18:47
@infirit infirit changed the title Use GAction where possible and add accelerators Use GAction where possible Jan 4, 2023
@sonarqubecloud
Copy link

sonarqubecloud bot commented May 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@github-actions github-actions bot added the stale label Feb 22, 2024
@github-actions github-actions bot closed this Mar 1, 2024
@cschramm cschramm removed the stale label Mar 1, 2024
@cschramm cschramm reopened this Mar 1, 2024
Copy link

sonarqubecloud bot commented Mar 1, 2024

Quality Gate Passed Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

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