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

Screen layouts can be overlapped #3842

Closed
1 task done
Shonkonray opened this issue Jul 22, 2023 · 2 comments · Fixed by #3845
Closed
1 task done

Screen layouts can be overlapped #3842

Shonkonray opened this issue Jul 22, 2023 · 2 comments · Fixed by #3845
Labels
bug 🪲 Something isn't working triage Bug report awaiting review / sorting

Comments

@Shonkonray
Copy link

Is there an existing issue for this problem?

  • I have searched the existing issues

What part of EdgeTX is the focus of this bug?

Transmitter firmware

Current Behavior

If during widget setup the button "TELE" on the tx (or companion simulator) is pressed the layout selection screen pops up. There a new layout can be selected and if then the "setup widgets" option is pressed the old and the new layout are overlapped.
This can be repeated many times and widgets can be selected until at some point the system crashes. If one wants to abort setting up widgets, he has to cycle back with the return button as many instances as were prior "opened" with the tele button.

Expected Behavior

I think best would be that the TELE button behaves like the RTN button, unless there is a good reason that it needs to be like this.

Steps To Reproduce

  1. Add telemetry screen
  2. Press setup widgets
  3. Press TELE button
  4. Press setup widgets
  5. Repeat as desired or until crash :)

Version

2.9.0

Transmitter

FrSky X10 / X10S (ACCST)

Operating System (OS)

Windows

OS Version

No response

Anything else?

Tested on 2.8.4 and 2.9.0_RC1, companion simulator behaves the same as TX.
screenshot_x10_23-07-22_17-41-06

@Shonkonray
Copy link
Author

Thank you for linking, and sorry that i have overlooked that there was already a pull request.

@pfeerick
Copy link
Member

pfeerick commented Jul 24, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🪲 Something isn't working triage Bug report awaiting review / sorting
Projects
None yet
2 participants