Skip to content

[BUG] Use of PopupMenuHandler in TooltipHandler #961

@chrisob194

Description

@chrisob194

Describe the bug

In the reset function inside the TooltipHandler, it is required to have the PopupMenuHandler plugin loaded in the graph, otherwise the tooltip will never show. PopupMenuHandler is required to check whether isMenuShowing is true. In my opinion it should be handled differently, for example by checking if isMenuShowing is true only when the PopupMenuHandler is present, but I don't know if this is an expected behaviour.

To Reproduce

Steps to reproduce the behavior:

  1. Use an instance of BaseGraph
  2. Add TooltipHandler plugin (without adding PopupMenuHandler)

Here it's a super basic stackblitz.
https://stackblitz.com/edit/typescript-demo-ilrtxwfk?file=index.ts

Check the comment at line 14

Expected behavior

Tooltips should be visible regardless of PopupMenuHandler, which covers another feature.

Screenshots

Environment

  • maxGraph version or commit: 0.21.0
  • Desktop or mobile: Desktop
  • OS and version: Linux Mint 22.2 Cinnamon
  • Browser and version: chrome 142
  • Node/npm version (if applicable): any
  • Used frameworks (if applicable): None

Additional context

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions