BBSListViewBase: Update context menu construction to use Gio::Menu #1003
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
サイドバーの右クリックメニューの構築を更新して廃止予定の`GtkAction`や `GtkUIManager`のかわりに`GAction`、`GtkBuilder`、`GMenu`を使った 方法にします。サイドバーの種類によっては使わないメニューがあるため 構築を分けることでメモリ使用量を削減します。合わせて、GTK4で廃止される `GtkMenu`への依存を減らすためメニューラベルにアクセラレーターキーや マウスジェスチャーの表示を追加する処理を削除します。
ma8ma
commented
Jun 18, 2022
|
|
||
|
|
||
| // GtkBuilderのUI definitionはメニューごとに分けて使うものだけ構築する | ||
| namespace { |
There was a problem hiding this comment.
UIがここで定義されていますが派生クラスに振り分けるほうがOOPの原則的にはすっきりするかもしれません
| popupmenu = id2popupmenu( "/popup_menu_select" ); | ||
| CONTROL::set_menu_motion( popupmenu ); | ||
| // アクショングループを作ってビューに登録 | ||
| m_action_group = Gio::SimpleActionGroup::create(); |
There was a problem hiding this comment.
GTKが一つのActionGroupを複数のwidgetで共有できる仕様ならビューをまとめるクラスに移動できます
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
サイドバーの右クリックメニューの構築を更新して廃止予定の
GtkActionやGtkUIManagerのかわりにGAction、GtkBuilder、GMenuを使った方法にします。サイドバーの種類によっては使わないメニューがあるため構築を分けることでメモリ使用量を削減します。
合わせて、GTK4で廃止される
GtkMenuへの依存を減らすためメニューラベルにアクセラレーターキーやマウスジェスチャーの表示を追加する処理を削除します。関連のissue: #977