Skip to content

Rework tooltips + complete them for relevant sections#1224

Merged
nikku merged 1 commit into
mainfrom
tooltip-reworks
May 22, 2026
Merged

Rework tooltips + complete them for relevant sections#1224
nikku merged 1 commit into
mainfrom
tooltip-reworks

Conversation

@nikku
Copy link
Copy Markdown
Member

@nikku nikku commented May 22, 2026

Proposed Changes

Applies a number of adjustments to the tooltips:

screenshot AyQNmU

Rules applied

  • Tooltips should be short and concise
  • Provide additional context information that cannot otherwise be sourced by field descriptions and other things
  • Every section that is not self-explainatory has a tooltip - backed by / referencing the documentation
  • Field level tooltips provide additional details - if needed

Try out

npx @bpmn-io/sr bpmn-io/bpmn-js-properties-panel#tooltip-reworks

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

Related to bpmn-io/properties-panel#499

@nikku nikku requested review from a team and Copilot May 22, 2026 08:33
@bpmn-io-tasks bpmn-io-tasks Bot added the needs review Review pending label May 22, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Reworks and extends Zeebe properties panel tooltips to provide more concise, documentation-backed guidance for groups and fields in the BPMN properties panel.

Changes:

  • Refined existing tooltip copy (e.g., assignment definition, condition, called decision link title).
  • Added/expanded tooltips for additional groups (e.g., headers, task/execution listeners, extension properties, output propagation, ad-hoc related groups).
  • Improved conditional-event variable events tooltip to avoid duplicated text while clarifying default behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/contextProvider/zeebe/TooltipProvider.js Outdated
@nikku nikku added the enhancement New feature or request label May 22, 2026
@nikku
Copy link
Copy Markdown
Member Author

nikku commented May 22, 2026

Find instructions to repeat the review here.

@nikku nikku requested review from Balazs-PM and karl-heinsen May 22, 2026 08:57
@nikku nikku changed the title feat: rework tooltips Rework tooltips + complete them for relevant sections May 22, 2026
@nikku nikku force-pushed the tooltip-reworks branch from 611b0ee to 439ad19 Compare May 22, 2026 09:12
@nikku nikku merged commit 2894276 into main May 22, 2026
8 of 9 checks passed
@nikku nikku deleted the tooltip-reworks branch May 22, 2026 11:54
@bpmn-io-tasks bpmn-io-tasks Bot removed the needs review Review pending label May 22, 2026
<div>
{ translate('Define the decision to evaluate and how to map back the evaluation result. ') }
<a href="https://docs.camunda.io/docs/components/modeler/bpmn/business-rule-tasks/#defining-a-called-decision" target="_blank" rel="noopener noreferrer" title={ translate('Send task documentation') }>
<a href="https://docs.camunda.io/docs/components/modeler/bpmn/business-rule-tasks/#defining-a-called-decision" target="_blank" rel="noopener noreferrer" title={ translate('Called decision documentation') }>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤦 on us

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants