チケット #42167

gui-gtk* : F2 units and F5 economy : add an empty column for scrollbar to prevent hidding last column content

登録: 2021-05-04 16:02 最終更新: 2021-05-22 03:14

報告者:
担当者:
チケットの種類:
状況:
完了
コンポーネント:
マイルストーン:
優先度:
5 - 中
重要度:
5 - 中
解決法:
修正済み
ファイル:
9

詳細

3.0beta2 (but i believe it is already in 2.6 )

with gtk3.2 client, the "F5" economy view does not always show the last column ' total' .

Everything is ok while names are short (as in the begining of the game) With buildings with long names such as aqueduc_lake (and maybe when they need two digit) , the last column is getting squeezed and becomes unreadable.

チケットの履歴 (37 件中 3 件表示)

2021-05-04 16:02 更新者: alain_bkr
  • 新しいチケット "gtk3.22 : F5 economy view : columns are not wide enough" が作成されました
2021-05-13 21:27 更新者: alain_bkr
コメント

The problems comes from the scrollbar which appears on the right side when the list is too big. The scrollbar overlaps the column, and thus hides its content.

The same thing happens for F2-units report.

(編集済, 2021-05-13 23:17 更新者: alain_bkr)
2021-05-13 22:54 更新者: alain_bkr
  • 添付ファイル fix_gtk322_F5_scrollbar_hiding_column_content.patch (File ID: 6797) が付加されました
2021-05-13 22:58 更新者: alain_bkr
コメント

patch adds an empty column for the scrollbar

Done on

commit c7ddb47e46c9acc73eefaaeb649824249d3902f6 (HEAD -> S3_0, origin/S3_0)
Date:   Thu Apr 22 04:54:14 2021 +0300

(編集済, 2021-05-13 23:17 更新者: alain_bkr)
2021-05-13 23:12 更新者: cazfi
コメント

Is gtk3-client affected?

In S3_1/master also gui-gtk-4.0 (gtk3x-client) should be patched (if trivial). Don't worry if you can't compile it, we don't expect contributors to test changes to it, just to provide same changes than are being made to gtk3.22-client.

2021-05-13 23:15 更新者: alain_bkr
  • 添付ファイル fix_gtk322_F5_scrollbar_hiding_column_content.patch (File ID: 6797) が削除されました
2021-05-13 23:16 更新者: alain_bkr
コメント

A patch fixing F2 (not F4 ) and F5 problem of scrollbar overlapping dat

2021-05-13 23:16 更新者: alain_bkr
  • 添付ファイル fix_F2_units_and_F5_economy_scrollbar_overlapping.patch (File ID: 6798) が付加されました
2021-05-13 23:22 更新者: alain_bkr
コメント

Reply To cazfi

Is gtk3-client affected? In S3_1/master also gui-gtk-4.0 (gtk3x-client) should be patched (if trivial). Don't worry if you can't compile it, we don't expect contributors to test changes to it, just to provide same changes than are being made to gtk3.22-client.

It is on gtk-3.22 client

I'll have a look at 3_1 / gtk-4 , the patch is rather simple so i am confident.

2021-05-14 03:05 更新者: alain_bkr
コメント
(このコメントは削除されました)
2021-05-14 03:06 更新者: alain_bkr
  • 添付ファイル fix_F2_units_and_F5_economy_scrollbar_overlapping.patch (File ID: 6799) が付加されました
2021-05-14 03:06 更新者: alain_bkr
  • 添付ファイル fix_F2_units_and_F5_economy_scrollbar_overlapping.patch (File ID: 6798) が削除されました
2021-05-15 21:58 更新者: alain_bkr
2021-05-16 03:03 更新者: alain_bkr
  • チケットの種類バグ から パッチ に更新されました
  • 概要が更新されました
2021-05-16 12:52 更新者: cazfi
コメント

Reply To alain_bkr

Reply To cazfi

Is gtk3-client affected? In S3_1/master also gui-gtk-4.0 (gtk3x-client) should be patched (if trivial). Don't worry if you can't compile it, we don't expect contributors to test changes to it, just to provide same changes than are being made to gtk3.22-client.

It is on gtk-3.22 client

but not in gtk3-client?
2021-05-16 12:58 更新者: cazfi
コメント

Reply To alain_bkr

I'll have a look at 3_1 / gtk-4 , the patch is rather simple so i am confident.

Most of the time you can fix all the gtk-clients by creating patch for one (for example gtk3.22), and then 'patch -u -p3 -d client/gui-gtk-3.0 < gtk3.22_fix.patch' & 'patch -u -p3 -d client/gui-gtk-4.0 < gtk3.22_fix.patch'

2021-05-17 22:44 更新者: alain_bkr
コメント

thanks for git tips :-)

I made 2 patches . one for 2_6 and 3_0 , concerning gtk3* one for 3_1 , concerning gtk3.22 and gtk4

2021-05-17 22:45 更新者: alain_bkr
  • 添付ファイル fix_F2_units_and_F5_economy_scrollbar_overlapping.patch (File ID: 6799) が削除されました
2021-05-17 22:45 更新者: alain_bkr
  • 添付ファイル S2_6__S3_0__gtk3.0_gtk3.22_fix_F2_units_and_F5_economy_scrollbar_overlapping.patch (File ID: 6838) が付加されました
2021-05-17 22:46 更新者: alain_bkr
  • 添付ファイル S3_1__gtk3.22_gtk4_fix_F2_units_and_F5_economy_scrollbar_overlapping.patch (File ID: 6839) が付加されました
2021-05-17 22:50 更新者: alain_bkr
コメント

thanks for git tips :-)

I made 2 patches .

  • one for 2_6 and 3_0 , concerning gtk3.0 and gtk3.22
  • one for 3_1 , concerning gtk3.22 and gtk4

They apply without problem , with zero or 1 line offset

2021-05-17 22:53 更新者: cazfi
コメント

Shouldn't it be 3 patches:

master: gtk3.22, gtk4

S3_1: gtk3.22, gtk3, gtk4

S3_0 & S2_6: gtk3.22, gtk3

2021-05-17 22:54 更新者: alain_bkr
  • 概要が更新されました
2021-05-17 23:31 更新者: alain_bkr
  • 添付ファイル S3_1__gtk3.22_gtk4_fix_F2_units_and_F5_economy_scrollbar_overlapping.patch (File ID: 6839) が削除されました
2021-05-17 23:31 更新者: alain_bkr
  • 添付ファイル S2_6__S3_0__gtk3.0_gtk3.22_fix_F2_units_and_F5_economy_scrollbar_overlapping.patch (File ID: 6838) が削除されました
2021-05-17 23:35 更新者: alain_bkr
コメント

There are too any branches and versions for my poor git knowledge. So I attached 3 patches, one for each gtk-version.

2021-05-20 07:27 更新者: cazfi
  • 解決法なし から 受領 に更新されました
コメント

I made them per-branch patches. As you had not made your patches with 'git format-patch', you were not set as the author in any case (it's going to show up me as the author, with credits to you in the commit message).

2021-05-22 03:14 更新者: cazfi
  • 状況オープン から 完了 に更新されました
  • 担当者(未割り当て) から cazfi に更新されました
  • 解決法受領 から 修正済み に更新されました

編集

このチケットにコメントを追加するには、ログインが必要です » ログインする