hop icon indicating copy to clipboard operation
hop copied to clipboard

[Bug]: Incorrect tooltip position

Open dave-csc opened this issue 1 year ago • 3 comments

Apache Hop version?

2.9.0

Java version?

openjdk 21.0.2 2024-01-16

Operating system

Linux

What happened?

When hovering a hop in Hop GUI, the tooltip is shown at the top left of the window, rather than next to the hop itself, see the example below:

Schermata del 2024-08-28 11-22-00

Using Ubuntu 22.04.4 on VirtualBox 7.20 in Windows 10 host, no particular graphics options enabled

Issue Priority

Priority: 3

Issue Component

Component: Hop Gui

dave-csc avatar Aug 28 '24 09:08 dave-csc

BEWARE OF THE FILE ABOVE, it's a malware! (already reported to GitHub)

dave-csc avatar Aug 28 '24 10:08 dave-csc

I have deleted the comment, thanks @dave-csc !

hansva avatar Aug 28 '24 11:08 hansva

Update: the same issue happens after upgrading the OS to Ubuntu 24.04.1 (with the same configuration for the other tools).

dave-csc avatar Sep 06 '24 09:09 dave-csc

Setting the GDK_BACKEND environment variable to x11 within the hop-gui.sh script (see screenshot below) can help resolve compatibility issues when running HOP on Wayland by forcing GTK-based applications to use X11 rendering.

Related issues:

  • https://github.com/apache/hop/issues/4619
  • https://github.com/eclipse-platform/eclipse.platform.swt/issues/639
  • https://github.com/eclipse-platform/eclipse.platform.swt/issues/790

image

ep9io avatar Dec 17 '24 16:12 ep9io

Patching the hop-gui.sh file indeed solves the issue, thanks @ep9io ✌️

dave-csc avatar Dec 18 '24 07:12 dave-csc

Update: this patch also solves #3783 and #3813 ✌️ ✌️

dave-csc avatar Dec 18 '24 13:12 dave-csc

fixed by #4720

hansva avatar Dec 18 '24 13:12 hansva

Can we change that to?

    if [ "${XDG_SESSION_TYPE}" = "wayland" ]; then
        GDK_BACKEND=x11
    fi

mattcasters avatar Dec 19 '24 10:12 mattcasters

GDK_BACKEND=x11

We need to have the export prior to the setting of the variable.

However, good point about wrapping the env param in quotes as that's good practice to avoid word splitting and empty variable substitution. The env param should be set correctly in most circumstances, but there's no harm with what you suggested.

ep9io avatar Dec 19 '24 10:12 ep9io