Positioning new windows based on the BOB display's x and y properties
メンテナーはふだん 2 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
Start with the cited DockItemRepresentation.java code and trace how command-line launches with target=window are handled. Compare that path with the ActionButton behavior for position and dimensions. The issue asks whether to change the default or add an opt-in preference, so the intended behavior needs agreement before implementation can be considered done.
索引モデルが issue の本文から書いたものです。
説明
When a display is launched from the command line and opened as a 'New Window', i.e. phoebus -resource file:/xx/xxx/file.bob?target=window, it will always be positioned at the default x=50, y=50 location on the monitor.
However, when you open a display from an ActionButton in the 'New Window' mode, it reads the x and y defined in the BOB file and positions that display in that location:
https://github.com/ControlSystemStudio/phoebus/blob/31059edc069e5d1df75202c1d6f31972e0a5b868/app/display/runtime/src/main/java/org/csstudio/display/builder/runtime/app/DockItemRepresentation.java#L65
The same is also true of width & height. Opening from an ActionButton will open the window with dimensions based on the BOB display's width and height properties:
https://github.com/ControlSystemStudio/phoebus/blob/31059edc069e5d1df75202c1d6f31972e0a5b868/app/display/runtime/src/main/java/org/csstudio/display/builder/runtime/app/DockItemRepresentation.java#L86
, while launching from a terminal using the default 1600 x 900 dimensions.
These two behaviors seem inconsistent. While I know that you can manually specify the width, height, x and y from the command line, I was wondering whether the default behavior when launching a new window from the terminal should be to position the new window at the x and y defined in the model and resize the window accordingly?
This was also what used to be done by CS-Studio when you opened with Position=NEW_SHELL in the command line argument.
Any thoughts on whether this would be a reasonable thing to do? It would impact the existing behavior so we may need to use an 'opt in' preference.
- 主要言語
- Java
- スター
- 130
- フォーク
- 132
- 平均マージ
- 2日 11時間
- マージ済み PR（30日）
- 19
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
ControlSystemStudio/phoebus のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
ControlSystemStudio/phoebus#3983 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
ControlSystemStudio/phoebus#3979 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
ControlSystemStudio/phoebus#3962 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
ControlSystemStudio/phoebus#3865 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 52/100
ControlSystemStudio/phoebus#3980 ·
メンテナーはふだん 2 日以内に返信
ControlSystemStudio/phoebus の issue をすべて見る
似ている issue
-
dev/release: verify-release-candidate.sh runs 0 Java tests with Debian's Maven対応中かも @kou が今日担当しました。 オープンType: bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
apache/arrow-adbc#4947 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
lysine-dev/okhttp#9814 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
checkstyle/checkstyle#22153 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 79/100
メンテナーはふだん 1 日以内に返信