search

EdgeTX / edgetx

EdgeTX is the cutting edge open source firmware for your R/C radio
https://edgetx.org
GNU General Public License v2.0
1.59k stars 338 forks source link

Merge 'Version' and 'About EdgeTX' #4161

Open philmoz opened 1 year ago

philmoz commented 1 year ago

Is there an existing issue for this feature request?

Is your feature request related to a problem?

It seems redundant to have both Version and About EdgeTX views with overlapping info.

Describe the solution you'd like

Suggest merging them together, accessed from the About EdgeTX button (remove Version tab from radio settings).

screenshot_el18_23-10-04_15-53-43

screenshot_tx16s_23-10-04_15-52-56

Any thoughts?

Describe alternatives you've considered

No response

Additional context

No response

pfeerick commented 1 year ago

I have no real preference for one or the other. Other than one being less detailed, and thus less information overload when you just want to know the version you are running, or the QR code for the website. You could argue that with it being on the splash screen that isn't necessary, but that only applies to a non-custom splash screen (and colorlcd 🤔). One thing that is conspicuously absent from the color UI which the B&W UI does have is the URL as a string... for those of use who are not glued to our smartphones ;)

The only think I will insist on is consistency... if it's done on colorlcd, it needs to be done for B&W also.

Message ID: @.***>

JimB40 commented 1 year ago

  1. We have a little space here so I'd add CODENAME under VERS if build is release or RC (same scheme as in splash)

  2. Since QR code is in color. Have you tested it low contrast themes? Does it work well? If not plain B&W with high contrast ratio will be better.