sarutak commented on PR #56694: URL: https://github.com/apache/spark/pull/56694#issuecomment-4780465411
@pan3793 ### Regarding the tooltip position I think it's better to keep it centered. The hint's purpose is to explain *why scrolling doesn't zoom* at the moment the user attempts it so it needs to appear where the user is already looking (i.e., near their cursor, which is typically over the center of the DAG). A hint placed in the corner (e.g., below the zoom toolbar) would likely go unnoticed because it's outside the user's focal area during scrolling. This follows an established UX pattern used by embedded Google Maps, Figma, and Miro. All show a centered, transient hint when the user scrolls without the required modifier key. That said, I've reduced the display duration from 1.5s to 1.0s (+ 0.2s fade-out) to minimize the visual intrusion. It's brief enough to feel like a flash rather than a persistent overlay. ### Regarding the SVG default sizing The 70vh fixed viewport was introduced in SPARK-56792. I think it deserves its own JIRA/PR since it involves different trade-offs (large vs. small query readability). Happy to discuss further in a separate issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
