Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Incorrect tooltip position #4252

Closed
dave-csc opened this issue Aug 28, 2024 · 9 comments
Closed

[Bug]: Incorrect tooltip position #4252

dave-csc opened this issue Aug 28, 2024 · 9 comments
Labels
Milestone

Comments

@dave-csc
Copy link
Contributor

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

This comment was marked as outdated.

@apache apache deleted a comment Aug 28, 2024
@hansva
Copy link
Contributor

hansva commented Aug 28, 2024

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

@dave-csc
Copy link
Contributor Author

dave-csc commented Sep 6, 2024

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

ep9io pushed a commit to ep9io/hop that referenced this issue Dec 17, 2024
@ep9io
Copy link
Contributor

ep9io commented Dec 17, 2024

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:

image

hansva added a commit that referenced this issue Dec 17, 2024
@dave-csc
Copy link
Contributor Author

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

@dave-csc
Copy link
Contributor Author

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

@hansva hansva added this to the 2.12 milestone Dec 18, 2024
@hansva
Copy link
Contributor

hansva commented Dec 18, 2024

fixed by #4720

@hansva hansva closed this as completed Dec 18, 2024
@mattcasters
Copy link
Contributor

Can we change that to?

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

@mattcasters mattcasters reopened this Dec 19, 2024
@ep9io
Copy link
Contributor

ep9io commented Dec 19, 2024

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 pushed a commit to ep9io/hop that referenced this issue Dec 19, 2024
hansva added a commit that referenced this issue Dec 24, 2024
@hansva hansva closed this as completed Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants