edgetx icon indicating copy to clipboard operation
edgetx copied to clipboard

Use GV name if assigned instead of GVx

Open jmxp69 opened this issue 2 years ago • 6 comments

Is there an existing issue for this feature request?

  • [X] I have searched the existing issues

Is your feature request related to a problem?

If we edit a global variable GVx and give it a name: FLT then add that global variable as a widget using Value that global variable is displayed on the screen as GVx and not FLT.

Describe the solution you'd like

I believe if a user assigns a name to a GV, the name should be used throughout. We do the same thing with Curves. If we use no curve name, CVx is used. If we assign a name, that name is used throughout.

Describe alternatives you've considered

Buying pfeerick a stack raffle tickets for EdgeTX Fest giveaways.

Additional context

No response

jmxp69 avatar Aug 18 '22 12:08 jmxp69

I'm afraid you'd probably have to do it in person, with a bunch of disguises and false names... given I'm part of the project my name won't be in the draw ;)

Oh, interesting... the name is shown when it's a GV only list ... image

... but not when it's a source list ... very interesting indeed, and not very consistent! 😆 image

Hm... this looks like where to fix it... and the nice thing is it should apply to B&W and Colorlcd radios if done there... I wonder if it would be better to show like with Companion in this instance... i.e so you know it's a GV...

image

https://github.com/EdgeTX/edgetx/blob/6648dffc8c89d34fd26aba544e507d3cf3b20e47/radio/src/strhelpers.cpp#L677-L678

pfeerick avatar Aug 19 '22 08:08 pfeerick

This just seems to leave it so you need to see an un-named one before you realise they're the GVs...

image

pfeerick avatar Aug 19 '22 09:08 pfeerick

I wonder if it would be better to show like with Companion in this instance... i.e so you know it's a GV...

This is what I would opt for, but is there enough screen space on the B&W radios?

gagarinlg avatar Aug 19 '22 10:08 gagarinlg

Yeah, that's what I was thinking afterward, so am going to avoid that problem for now. 😆 On B&W they'll just have to continue to suffer with GV# as the UI there changes in place, rather than via a popup list.

pfeerick avatar Aug 19 '22 10:08 pfeerick

I'm afraid you'd probably have to do it in person, with a bunch of disguises and false names... given I'm part of the project my name won't be in the draw ;)

AWWW - I'll just have to find my kryptonite elsewhere I guess.

Hm... this looks like where to fix it... and the nice thing is it should apply to B&W and Colorlcd radios if done there... I wonder if it would be better to show like with Companion in this instance... i.e so you know it's a GV...

image

https://github.com/EdgeTX/edgetx/blob/6648dffc8c89d34fd26aba544e507d3cf3b20e47/radio/src/strhelpers.cpp#L677-L678

Why does your curve 2 and 3 have 4 characters hmmmm????? I saw that....

jmxp69 avatar Aug 19 '22 11:08 jmxp69

Hi Peter, I don't mind if the user-defined name is prefixed with GV: when it appears in a drop down list, but would prefer to display only the user-defined name when it is used in a widget for example. At that point I don't need to know that the displayed value's source is a GV, I just want it to have a meaningful label, preferably with up to 4 chars. Cheers, Geoff.

gjhaynes avatar Aug 19 '22 15:08 gjhaynes