Skip to content

Improve direct editing for events / start outside of sub-process#2416

Draft
nikku wants to merge 3 commits into
developfrom
748-fix-activation-start-event
Draft

Improve direct editing for events / start outside of sub-process#2416
nikku wants to merge 3 commits into
developfrom
748-fix-activation-start-event

Conversation

@nikku
Copy link
Copy Markdown
Member

@nikku nikku commented Apr 17, 2026 •
edited
Loading

Proposed Changes

This consistently enables direct editing for all elements but gateways (#748).
capture UbS3an_optimized

Note

Draft, because of the UX impact ("one more click") and validation pending.

What's inside

  • Simplify tests
  • Fix direct editing not activating for start events outside of sub-processes

Checklist

Ensure you provide everything we need to review your contribution:

  • Contribution meets our definition of done
  • Pull request establishes context
    • Link to related issue(s), i.e. Closes {LINK_TO_ISSUE} or Related to {LINK_TO_ISSUE}
    • Brief textual description of the changes
    • Screenshots or short videos showing UI/UX changes
    • Steps to try out, i.e. using the @bpmn-io/sr tool

@bpmn-io-tasks bpmn-io-tasks Bot added the in progress Currently worked on label Apr 17, 2026
This was referenced Apr 17, 2026
Open
Merged
@nikku nikku force-pushed the 748-fix-activation-start-event branch from 02124bb to 6b14882 Compare April 17, 2026 11:58
nikku added 3 commits April 20, 2026 14:18
@nikku
test(label-editing): generalize createElement to be re-usable
a4ce2e0 
* Can create in different parents
* Validate that parent exists
@nikku
test(label-editing): improve test diagram layout
c8ac7f9 
Make the diagram more readable with a little more space.
@nikku
feat(label-editing): activate label editing consistently for most ele…
26222e8 
…ments

As per our `label-required` rule start events should have labels. Let
us encourage them. If we feel like labels inside of sub-processes are
not required (as implemented in this PR) then it should be reflected
in the `label-required` rule.

Related to #748
Related to #2408
@nikku nikku force-pushed the 748-fix-activation-start-event branch from 6b14882 to 26222e8 Compare April 20, 2026 12:18
@nikku nikku added ux backlog Queued in backlog and removed in progress Currently worked on labels Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees

@nikku nikku

Labels

backlog Queued in backlog ux

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@nikku