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

Burrito icon overlaping tradingpost #219

Closed
Masgalor opened this issue Nov 13, 2023 · 5 comments
Closed

Burrito icon overlaping tradingpost #219

Masgalor opened this issue Nov 13, 2023 · 5 comments
Labels
bug Something isn't working duplicate This issue or pull request already exists user interface An issue that deals with the UI

Comments

@Masgalor
Copy link
Contributor

I've installed the latest release available here on Github (Alpha 1.4) and noticed that the burrito icon overlaps the tradingpost icon.
This confuses me because there is already a merged PR to fix this #156 .
I'm not sure what to do now, do I need to build burrito myself from master to get this fix?

@AsherGlick
Copy link
Owner

This issue should be in the most recent stable release, that release was cut specifically for the issue. If there is a bug and it is not working for some reason then we will try to address it.

Can you attach:

  • A screenshot of this issue with no menus open (and no red outline around burrito)
  • A screenshot of this issue with at least one menu open (with a red outline around burrito)
  • What in-game UI scale you play on
  • How you are launching burrito link (d3d11.dll / standalone executable / something else)
  • The output to the commands inxi -Gxxx and wmctrl -m (just in case)

@Masgalor
Copy link
Contributor Author

Masgalor commented Nov 13, 2023

Maybe it's worth mentioning, that everything else (e.g. range indicator or markers) is where it is supposed to be.

I'm launching the burrito-link standalone executable with a script generated bey lutris as explained in the instructions.
Edit: Using the dll dosen't change anything.

  • Interface size: big
  • DPI-Scaling: disabled

1
2

user@localhost:~> inxi -Gxxx
Graphics:
  Device-1: AMD Navi 10 [Radeon RX 5600 OEM/5600 XT / 5700/5700 XT]
    vendor: Sapphire driver: amdgpu v: kernel arch: RDNA-1 pcie: speed: 16 GT/s
    lanes: 16 ports: active: DP-2,DP-3 empty: DP-1,HDMI-A-1 bus-ID: 0b:00.0
    chip-ID: 1002:731f class-ID: 0300
  Display: x11 server: X.Org v: 21.1.9 with: Xwayland v: 23.2.2
    compositor: kwin_x11 driver: X: loaded: amdgpu
    unloaded: fbdev,modesetting,radeon,vesa dri: radeonsi gpu: amdgpu
    display-ID: :0 screens: 1
  Screen-1: 0 s-res: 5120x1440 s-dpi: 96 s-size: 1354x381mm (53.31x15.00")
    s-diag: 1407mm (55.38")
  Monitor-1: DP-2 mapped: DisplayPort-1 pos: primary,right model: VG27B
    serial: L6LMQS029772 res: 2560x1440 dpi: 109 size: 597x336mm (23.5x13.23")
    diag: 685mm (27") modes: max: 2560x1440 min: 720x400
  Monitor-2: DP-3 mapped: DisplayPort-2 pos: left model: VG27B
    serial: M4LMQS275540 res: 2560x1440 dpi: 109 size: 597x336mm (23.5x13.23")
    diag: 685mm (27") modes: max: 2560x1440 min: 720x400
  API: OpenGL v: 4.6 Mesa 23.2.1 renderer: AMD Radeon RX 5700 XT (navi10
    LLVM 17.0.3 DRM 3.54 6.5.9-1-default) direct-render: Yes
user@localhost:~> wmctrl -m
Name: KWin
Class: N/A
PID: N/A
Window manager's "showing the desktop" mode: N/A

@AsherGlick
Copy link
Owner

From this info there is a 99% chance it is Interface size: big . Values other then normal is something we have not implemented yet, though it is entirely possible to do so and we plan to do this eventually.

If you are feeling ambitious you can hack this for yourself. All you will need to do is reexport the overlay executable itself after making an edit, you can reuse the other libraries you already downloaded without rebuilding them. I think you will need to adjust this line but I am not 100% sure about that. Here are the instructions to export it yourself entirely through the command line. You can also just download godot 3.3.2 and export it manually, all the export settings are preconfigured and there are instructions online for how to export godot projects in general.

Our priority right now is trying to finish #64 by the end of the year, so dont expect much from us in way of prioritizing this before then. That said, I will happily review a pr for dynamic ui-size adjustments from you or anyone else. The value that will need to be read is uisz inside the json identity field from the mumble api. iirc we are actually already sending that data over via mumble link, but I dont know what all will need to be adjusted based on that setting.

@AsherGlick AsherGlick added user interface An issue that deals with the UI bug Something isn't working labels Nov 13, 2023
@AsherGlick
Copy link
Owner

I am going to close this as a duplicate of #16 . If this is not actually the issue feel free to re-open it. If you have any quick questions about this manual fix I can try to help but I wont be able to walk you through it more then the guidance above, someone else might though.

@AsherGlick AsherGlick added the duplicate This issue or pull request already exists label Nov 13, 2023
@Masgalor
Copy link
Contributor Author

You are absolutely right, the problem here is that the burrito icon doesn't adjust itself to the interface scale.
I really appreciate your detailed answer to this issue, I had no problems to write a patch that places the icon dynamically depending on the interface scale.

I will ope a draft PR to see if you think it's usable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists user interface An issue that deals with the UI
Projects
None yet
Development

No branches or pull requests

2 participants