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

[IDEA] Imageviewbase update menu construction #66

Closed
wants to merge 2 commits into from

Conversation

ma8ma
Copy link
Owner

@ma8ma ma8ma commented Jul 17, 2022

このPRはアイデア検証用でありマージしません。

[IDEA] Usrcmd_Manager: Update context menu construction to use Gio::Menu

ユーザーコマンドのメニュー構築を更新して廃止予定のGtkActionGtkActionGroupのかわりにGActionGSimpleActionGroupGMenuを使った方法にします。
usrcmdを使うビューをすべて更新するまで従来の関数は削除しません。

[IDEA] ImageViewBase: Update context menu construction to use Gio::Menu

画像ビューの右クリックメニューの構築を更新して廃止予定のGtkActionGtkUIManagerのかわりにGActionGtkBuilderGMenuを使った方法にします。
合わせて、GTK4で廃止されるGtkMenuへの依存を減らすためメニューラベルにアクセラレーターキーやマウスジェスチャーの表示を追加する処理を削除します。

変更点

  • GAction, GMenuではサブメニューを無効化できないためサブメニュー内の項目に対して選択可能か設定する

関連のissue: JDimproved#977

ユーザーコマンドのメニュー構築を更新して廃止予定の`GtkAction`や
`GtkActionGroup`のかわりに`GAction`、`GSimpleActionGroup`、`GMenu`を
使った方法にします。
usrcmdを使うビューをすべて更新するまで従来の関数は削除しません。
画像ビューの右クリックメニューの構築を更新して廃止予定の`GtkAction`や
`GtkUIManager`のかわりに`GAction`、`GtkBuilder`、`GMenu`を使った
方法にします。合わせて、GTK4で廃止される`GtkMenu`への依存を減らすため
メニューラベルにアクセラレーターキーやマウスジェスチャーの表示を
追加する処理を削除します。

変更点
- GAction, GMenuではサブメニューを無効化できないため
  サブメニュー内の項目に対して選択可能か設定する
@ma8ma ma8ma added the archive label Jul 17, 2022
@ma8ma ma8ma changed the title Imageviewbase update menu construction [IDEA] Imageviewbase update menu construction Jul 17, 2022
@ma8ma
Copy link
Owner Author

ma8ma commented Jan 8, 2023

検証を終えたため閉じます。

@ma8ma ma8ma closed this Jan 8, 2023
@ma8ma ma8ma deleted the imageviewbase-update-menu-construction branch January 8, 2023 11:41
@ma8ma ma8ma added the wontfix This will not be worked on label Aug 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
archive wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant