Added notification when no GPSdata is received

[rotorman]

Also removed the shadow effect on font to make text better readable.

[pfeerick]

Does anything need doing to this? Otherwise, I can merge when ready.

[rotorman]

I hope this to be mergeable as is ;)

[pfeerick]

Sold! Just wanted to give you a chance to revisit it since it has been a little while! 😁 Actually, now that I'm reviewing it, I think it needs a little TLC... it looks like the if for the else is commented out... 😖

[rotorman]

The else at line 56 is for if at line 36:
https://github.com/rotorman/edgetx-sdcard/blob/e046dbdef2029db58127d1a067504f126c9d731d/sdcard/horus/WIDGETS/TxGPStest/main.lua#L36

[pfeerick]

Yeah, I've just redone the indentation as it was all over the place, so can see it now. Thanks! :)

Do you think this is better, or leave it as it is? (Widget is in two separate panels - big and small - hence the duplication)

[rotorman]

Awesome thanks! Different editors have different indentation... on mine locally the old was also OK, but I see the bad indentation issue here in GH on my original commit.

The text addition is good - thanks! The reason could also be that the GPS support is enabled in firmware, but the GPS is not connected. What do you say to:

Make sure you have connected GPS and your firmware has GPS support enabled!

?

[pfeerick]

I added more text - the second and third lines ;) Unless simu acts differently to actual hardware in this instance - building with INTERNAL_GPS gives the following...

There is still space for at least another line of text, so can add it anyway...

[rotorman]

Aah, OK, so no detection of physical presence of GPS, just if build option is enabled or not - I get it now.
Thus your original text addition was perfect. Let's leave it at that then.
Thanks!

[pfeerick]

Final test on actual hardware matches simu (re: INTERNAL_GPS / no INTERNAL_GPS behaviour), so unfortunately is that dumb... would have to work off the fix key instead, which isn't so useful... "Either your GPS doesn't have fix yet, or you haven't configured it, it ain't compiled in, or it ain't plugged" ... not so useful! 😆