Skip to content

Make behaviour of Trust Boundary Boxes more reasonable#1724

Draft
boomer41 wants to merge 2 commits into
mainfrom
fix/box-handling
Draft

Make behaviour of Trust Boundary Boxes more reasonable#1724
boomer41 wants to merge 2 commits into
mainfrom
fix/box-handling

Conversation

@boomer41

Copy link
Copy Markdown
Collaborator

Summary:

Fixes #1723.
Split from #1718

When accidentally mis-clicking to an empty space, the box gets selected.
To un-select it I have to scroll to somewhere where the box is not, click on an empty space, and return.
I can not select any other node inside the box until the box is deselected.

Additionally, the left margin of the box label seems a bit "random".
It is somehow floating, but not centered nor fixed to any edge.

Description for the changelog:

  • Trust Boundary Boxes will no longer get selected when clicking on an empty space inside the box. To select a box, click on the border.
  • Aligned the label of Trust Boundary Boxes on the top left side of the box

Declaration:

Thanks for submitting a pull request, please make sure:

  • content meets the license for this project
  • appropriate unit tests have been created and/or modified
  • you have considered any changes required for the functional tests
  • you have read the contribution guide and agree to the Code of Conduct
  • either no AI-generated content has been used in this pull request
  • or any use of AI in this pull request has been disclosed below:
    • AI Tools: [e.g. GitHub CoPilot, ChatGPT, JetBrains Junie, etc]
    • LLMs and versions: [e.g. GPT-4.1, Claude Haiku 4.5, Gemini 2.5 Pro, etc]
    • Prompts: [Summarize the key prompts or instructions given to the AI tools]

Other info:

@boomer41 boomer41 self-assigned this Jun 10, 2026
@jgadsden

Copy link
Copy Markdown
Collaborator

This is a change to the behavior of the trust boundary boxes and needs discussion and thinking through, so putting ths pull request into draft

The existing trust boundary box behavior is as designed / expected, but I understand totally that it is difficult to suit every user, hence discussion is needed before a silent change like this is introduced

@jgadsden jgadsden marked this pull request as draft June 11, 2026 09:28
@github-actions

github-actions Bot commented Jul 5, 2026

Copy link
Copy Markdown

This PR is stale because it has been open 21 days with no activity. Remove stale label, or add a comment, otherwise it will be closed in 7 days

@github-actions github-actions Bot added the Stale label Jul 5, 2026
@boomer41

boomer41 commented Jul 5, 2026

Copy link
Copy Markdown
Collaborator Author

bad bot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Trust Boundary Boxes are interfering with normal workflow

2 participants