Skip to content

WIP move to Gio.Action for ManagerDeviceMenu#2820

Draft
infirit wants to merge 5 commits intoblueman-project:mainfrom
infirit:managerdevicemenuactions
Draft

WIP move to Gio.Action for ManagerDeviceMenu#2820
infirit wants to merge 5 commits intoblueman-project:mainfrom
infirit:managerdevicemenuactions

Conversation

@infirit
Copy link
Contributor

@infirit infirit commented Aug 2, 2025

No description provided.

@infirit infirit force-pushed the managerdevicemenuactions branch 3 times, most recently from 733ead1 to 7cc4e74 Compare August 2, 2025 15:31
No functional changes

* give variables better names
* don't shadow variable from outerscope (dialog)
* only set labels and iconnames variables in if/else blocks
* use_underline sets mnemonic on the label
* ImageMenuItem without can be created without image data
@infirit infirit force-pushed the managerdevicemenuactions branch from 7cc4e74 to 7911167 Compare August 23, 2025 12:10
@sonarqubecloud
Copy link

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.

1 participant