Add gtk3 support [1/2] #30
Merged
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
LLVMスタイルを元にJDスタイルを再現する
表記ゆれのある部分はJDのコードを調べて多数の方を採用する
曖昧な部分はLLVMスタイルを踏襲する
JDスタイルから変更した部分
* 単独のジャンプ文(continue, break, goto, return, throw)を除き
if-else文のブロックに波括弧を付ける
```cpp
// YES
if( expr ) return 1;
if( long_long_expr1 && long_long_expr2 )
return long_long_func( long_long_value );
if( expr ) {
var = 20;
}
// NO
if( expr ) var = 20;
if( expr )
var = 20;
```
* 空の文(;)を除きループ文のブロックに波括弧を付ける
```cpp
// YES
while( expr )
;
while( expr ) {
var += 1;
}
// NO
while( expr );
while( expr ) var += 1;
while( expr )
var += 1;
```
ツールで再現できていないスタイル
* マクロGTKMM_CHECK_VERSION()の引数
-> スペースを取り除いてJDスタイルに準ずる
```cpp
// YES
#if GTKMM_CHECK_VERSION(2,10,0)
#endif
// NO (clang-format)
#if GTKMM_CHECK_VERSION( 2, 10, 0 )
#endif
```
* 制御構文の条件式の閉じ括弧とブロックの開き波括弧の間
-> ツールで整形したままでよい
```cpp
// OK (clang-format)
if( expr ) {
}
// (JD style)
if( expr ){
}
```
* 否定演算子の後
-> ツールで整形したままでよい
```cpp
// OK (clang-format)
func( !expr );
// (JD style)
func( ! expr );
```
* for文の条件式の初期化パートが空のとき
-> ツールで整形したままでよい
```cpp
// OK (clang-format)
for( ; expr1; expr2 ) {
}
// (JD style)
for(; expr1; expr2 ){
}
```
gtkmm-2.4の代わりにgtkmm-3.0を使ってビルドするconfigureフラグを追加する gtkmm-3.0のバージョンは3.18以上を推奨 見出しや箇条書きなどを除きREADME.mdの文章をですます調にする
gtk3 removes many implementation details and struct members from public headers. 挙動の変更 リセットフラグを設定できないため即時リセットするように変更する
Gtk::Widget::on_expose_event() is removed.
Gdk::Drawable API is deprecated. cairomm 1.12.0 がメモリリークを起こしたので C API を使うことで 問題を回避する
コードの重複とプリプロセッサの分散を減らす
コードの重複とプリプロセッサの分散を減らす
Expose event signal is removed since gtk3.
Gtk::Widget::on_expose_event() is changed to on_draw().
条件コンパイルでGDK_<keyname>をGDK_KEY_<keyname>に置き換えると コードが重複して読みにくいので古い名前を使う gtk 2.22.0未満のサポートを打ち切れば条件コンパイル無しで置き換えられる
gtk3からGtk::Notebookのswitch-pageシグナルハンドラの 引数GtkNotebookPage*がGtk::Widget*に変更されたので GtkNotebookPageがGtk::Widgetの別名になるようにusingを追加する
* The parameter type of Gdk::Window::set_cursor() becomes shared smart pointer type. * The parameter type of Gtk::FileChooserDialog::add_filter() becomes shared smart pointer type.
GDK_WINDOW_XWINDOW() is removed since gtk3.
Gtk::Widget::render_icon() is renamed to render_icon_pixbuf() since gtk3. render_icon()の呼び出し箇所に条件コンパイルでrender_icon_pixbuf() の呼び出しを追加してくと差分が大きくなりコードが読みづらくなる そのためgtk3版ではローカル変数にrender_icon()を実装する
Gdk::Display::get_pointer() is removed since gtk3.
…rea() gtkmmのAPIを使わない理由 Gdk::Window::get_update_area()の戻り値のスマートポインター Cairo::RefPtr<Cairo::Region>をboolに評価しようとすると segmentation faultする この結果、スレビューをスクロールしようとしたときにJDがクラッシュした
Gtk::Widget::shape_combine_mask() is removed since gtk3. NOTE: アルファチャンネルが利用できない環境では背景を透過できない
Gtk::MenuItemにGtk::HBoxを追加する方法は動作しなくなった
* Use Gtk::Widget::override_font() instead of modify_font() * Use Gtk::StyleContext to get label font instead of Gtk::Style * Use Gtk::StyleContext to get entry font instead of Gtk::Style
* Use Gtk::Widget::override_color() instead of modify_fg() * Use Gtk::Widget::override_color() instead of modify_text() * Use Gtk::Widget::override_background_color() instead of modify_bg() * Use Gtk::Widget::override_background_color() instead of modify_base() * Use Gtk::Widget::unset_color() instead of unset_fg() * Use Gtk::StyleContext to get entry color instead of Gtk::Style * Use Gtk::StyleContext to get bg color instead of Gtk::Style ウィジェットの配色ではなくGTKテーマの設定から色を取得するように 変更した箇所がある
* Use Gtk::Widget::set_margin_xxx() instead of Gtk::(Rc)Style::set_(x|y)thickness() * Use Gtk::Widget::get_margin_xxx() instead of Gtk::(Rc)Style::get_(x|y)thickness()
DragableNoteBookの主なロジックのうち以下をgtk3でも動くように変更する * タブの描画は、自前で描画するのをやめてデフォルトのハンドラを使う * タブ幅の計算は、タブの代わりにラベルの領域を使う 既知の問題 * DnDでタブを並べ替えるときに表示する目標の矢印はタブの境界からずれる * DnDのドロップ範囲がタブの領域より狭くなっている * タブの上でマウスホイールするとページが切り替わる機能は 先頭<->最後尾の場合を除いて機能しない * ラベルの領域からシグナルを拾うのでタブの端付近では ドラッグを開始できない 既存のロジックは実装の詳細をgtkのソースコードから引用して使うことで タブの領域を取得する。gtk3ではデータ構造の変更[1]により今までの方法が できなくなった。 [1]: gtk 3.20から_GtkNotebookPageは割り当てられた領域(GtkAllocation) を持たなくなった。 https://gitlab.gnome.org/GNOME/gtk/blob/gtk-2-24/gtk/gtknotebook.c#L148 https://gitlab.gnome.org/GNOME/gtk/blob/gtk-3-24/gtk/gtknotebook.c#L293
タブDnDの目標矢印が境界からずれる＆ドロップ範囲がタブの領域より 狭くなっている不具合を低減する。GTKテーマによってはドロップ先を示す 矢印の描画に問題が発生する可能性がある。 XXX: この修正はラベルの左右の余白の大きさが同じであることを前提とする
* Omit DragableNoteBook::draw_box() and get_alloc_notebook(). * Omit custom draw for DragableNoteBook, TabNotebook TabSwitchButton, ToolBarNotebook and ViewNotebook. 枠の描画や座標取得のメソッドを省略しDragableNoteBookの描画処理は 自前でやらない 変更の理由 描画処理に必要な座標はgtkのソースコードを引用して実装の詳細に アクセスして取得している。gtk3ではデータ構造が変更された影響で 今までのコードが使えなくなった。
gtk3ではツールバーのカスタム描画をしない。 gtk2のJDToolbarの描画処理はgtkのソースコードを引用することで 実装の詳細にアクセスして行っている。gtk3ではデータ構造が 変更された影響で今までのコードが使えなくなった。 さらにGTKテーマによってはツールバーの子ウィジェットの配色がテーマに 従わないことがある。ツールバーからcssクラスGTK_STYLE_CLASS_TOOLBARを 削除することで配色を修正する。
style-changed signal is removed since gtk3. スタイルを反映するシグナルハンドラを省略する GTK3版ではGtkNotebook in GtkNotebookの背景色は問題がないように見える
gtk3 removes many implementation details and struct members from public headers. ハックによる枠の描画は省略する
GTK+ 3.9.2からGTK/GDKのダブルバッファ処理が再構成された [1] この影響でスレビューのスクロール時に画面のちらつきが発生するようになった ちらつきを抑えるため3.9.2以降の環境ではデフォルトのダブルバッファを使う [1]: https://gitlab.gnome.org/GNOME/gtk/commit/d22fd7223c75f4720ddb982c659efb0d8d7543c4
デフォルトの設定ではウインドウの幅が狭いとエントリーが表示されない
gtk3では伸縮するウィジェットを持つダイアログを表示したときに ウインドウのサイズが小さい場合があるためデフォルトのサイズを 設定するための機能を追加する
Gtk::POLICY_NEVERを設定するとスクロール機能自体がなくなり コンテンツに合わせてGtk::ScrolledWindowのサイズが拡大してしまう
gtk3はマウスホイールによるGtkNotebookのタブの切り替えが 動作しなくなったのでシグナルハンドラを使って再現する 未修正の問題 この修正に関係なくホイールが全く反応しない環境がある(>= gtk 3.20?) が厳密な条件は分からない
gtk3 removes many implementation details and struct members from public headers. XXX: フラット表示は装飾がない前提でcssが設定されている * Fix compile error for debug log. Thanks to @ooinu. Co-authored-by: ooinu <45452777+ooinu@users.noreply.github.com>
overrideメソッドの代わりにcssの設定で書き込みビューの配色を設定する また、書き込みビューの配色にGTKテーマを使う設定を合わせて追加する
overrideメソッドでは配色の変更が上手くいかない環境があるので cssを使って変更する。
ステータスバーのテキスト部分が上手く配色されないGTKテーマがあるので 明示的に設定する
GDK needs a transient parent window to display a popup window. [gtk+] wayland: prefer subsurface when possible https://mail.gnome.org/archives/commits-list/2016-January/msg02035.html
|
part1とありますが、あとどれくらいでしょうか。 |
|
@yama-natuki 今回の追加(#30)とタッチスクリーン対応で２回の予定です。(バグがなければ)今回の変更で基本的な部分はできています。 edit: タイトル修正しました 以下個人の感想です まあいいかと思う部分
さすがにどうかなと思う部分
|
|
ありがとうございます。 |
|
@naniwaKun GTK3版がmasterブランチにマージされました。ma8maのほうのgtk3-previewブランチはタッチスクリーン対応のテストに使いますからAURパッケージはmasterブランチに切り替えて差し支えないと思います。 |
|
@ma8ma |
|
@ma8ma |
|
重ねてスレットにレスを残しておきますね。 |
ma8ma
added
gtk3
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.
GTK3版のサポートを追加します。詳細はREADME.mdにも書いてあります。
追加された内容
GTK2版の変更はありません。
--with-gtkmm3を使うことでgtkmm2のかわりにgtkmm3を使ってビルドするGTK3版の要件
GTK3版の制限
--with-sessionlib=gnomeuiと--with-gtkmm3を指定するとコンパイルエラーになるコミットの概要
変更箇所が多いので要点をつまんでリストします。何か見落としている点があるかもしれないのでレビューをお願いいたします。
edit(2019-02-03): コミットのリンクを追加
--with-gtkmm3の追加とREADME.mdの修正ma8ma@6634431
ma8ma/JDim@ma8ma:1673167d75...ma8ma:5f7fcc1880
ma8ma@1673167
ma8ma/JDim@ma8ma:5f7fcc1880...ma8ma:634769c946
ma8ma/JDim@ma8ma:634769c946...ma8ma:af51da0ea4
ma8ma@e2fc23e
ma8ma/JDim@ma8ma:af51da0ea4e...ma8ma:43a6795a34