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

Fix maximum symbol. #160

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix maximum symbol. #160

wants to merge 1 commit into from

Conversation

her01n
Copy link

@her01n her01n commented Dec 7, 2024

On Taranis, it is possible to switch between displaying actual and maximum values, using the rotary control. The symbol is displayed to signify the values are maximum/minimum. This is implemented with drawing text with special characters "\192" and "\193".

On my version and hardware, edgetx v2.10.5, opentx-telemetry-widget 2.2.5, transmitter taranis qx7 access, the characters are not displayed - the space on display is blank.

I propose to replace the special characters with up and down arrow drawed with "lcd.drawLine" and "lcd.drawPixel".

>
> Replace "\192" and "\192" symbols with drawed up and up arrows.
@stronnag
Copy link
Collaborator

Have you tested this on OpenTX ?

The reason for the issue is because EdgeTX changed the fonts (to a UTF-8 font for localisation).
Have you checked if there is a a character ine the Edge font that can be used instead of a custom icon?

@her01n
Copy link
Author

her01n commented Dec 14, 2024

The bug is not present on OpenTX 2.3.15. The fixed version works with OpenTX with no visual difference.

The character is not available in EdgeTX, or at least is not documented in here https://luadoc.edgetx.org/appendix/fonts.
Left and right arrows are available, as are characters '^', 'v'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants