Widget alignment in editor: Use first widget or outline of all widgets?
维护者通常 2 天内回复
还没有人认领这个 Issue。
评估
调研方向
Start by comparing the behavior described in PR #509 with its imported version, PR #579, then locate the display editor’s alignment actions. The issue presents several competing options but does not choose one, so agree on the intended behavior before implementation; done means the chosen alignment behavior is clear and matches the rectangle-and-label example.
由索引模型根据 Issue 内容生成。
描述
The display editor alignment options were initially relative to the first selected widget.
Select a widget, add other widgets to the selection, and when you then align left/right/center, all widgets would be aligned to the left/right/center of the first selected widget.
In https://github.com/kasemir/org.csstudio.display.builder/pull/509 (later imported to phoebus as https://github.com/ControlSystemStudio/phoebus/pull/579), that behavior was changed.
Now left/right/center aligns all selected widget to the left/right/center of the combined widget outline.
The original behavior is more straight forward for precision alignment.
Select a widget that's already in the desired location, then select others and align them to the left/right/center of the first one.
Example:
Create a rectangle, place it as desired.
Create a Label, set text.
Select rectangle, then label, "Align center" and the text will be in the center of the rectangle.
Downside of the original behavior: Nobody knows how to select one widget, then add others to the selection. Need to hold control, command, alt or shift? Much easier to rubber band around all the widgets of interest and align to the outline. But now it's harder to truly align things.
Example:
Create a rectangle, place it as desired.
Create a Label, set text.
Rubberband-select rectangle and label
"Align center" and BOTH widgets will move to the center of their combined outline, so rectangle is no longer where you wanted it.
Options:
Keep as is. Alignment is slightly useless, but I never use it anyway?
Restore original alignment relative to first selected widget, and nobody will remember how to use that?
Remove alignment options to simplify user interface?
- 主要语言
- Java
- 星标
- 130
- 派生
- 132
- 平均合并
- 3 天 1 小时
- 30 天内合并 PR
- 24
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 没有贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
ControlSystemStudio/phoebus 的其他 Issue
-
难度 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 天内回复
-
难度 4/5 3-5 天 新手友好度 45/100
ControlSystemStudio/phoebus#3978 ·
维护者通常 2 天内回复
查看 ControlSystemStudio/phoebus 的全部 Issue
相似的 Issue
-
bug
难度 2/5 1-3 小时 新手友好度 82/100
adorsys/keycloak-config-cli#1709 ·
维护者通常 1 天内回复
-
! and-bugs
难度 2/5 1-3 小时 新手友好度 78/100
osmandapp/OsmAnd#26769 · 1 条评论 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 70/100
NationalSecurityAgency/ghidra#9997 · 2 条评论 ·
维护者通常 2 天内回复
-
[Bug] DefaultSerializationExceptionWrapper does not implement Cleanable, so cleanup() is never invoked by codecs可能已有人在做 @binbin0325 今天认领。 未关闭component/need-triage type/need-triage
难度 2/5 1-3 小时 新手友好度 75/100
维护者通常 1 天内回复
-
fix: tagged PDF fails with ClassCastException when a Form and its Image share an XObject name可能已有人在做 @hnc-jglee 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 78/100
opendataloader-project/opendataloader-pdf#830 ·
维护者通常 1 天内回复