Github user mcgilman commented on the issue:
https://github.com/apache/nifi/pull/1430
@jvwing Thanks for the prompt review!
I certainly don't mind having a more comprehensive review of the current
behavior. The initial stab at the PR was simply trying to address the confusing
bit reported surrounding the incorrect hand pointer over the event text.
We should probably consider updating the main canvas as well to ensure its
consistent throughout the application. In the context of the canvas, panning is
supported by clicking and dragging on the background and moving is supported by
clicking and dragging on a component. I'm thinking we should leave the arrow
cursor in place on the canvas (as it's currently implemented) and introduce the
move cursor when dragging (moving a component and panning only). This behavior
would be similar to other tools like Google Maps.
Carrying this behavior into the lineage graph, we could always have the
arrow pointer (like the canvas) and clicking the background would pan and show
the move cursor. Additionally, I think we should prevent panning if the
mousedown happens on a lineage node or event type text.
Thoughts?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
