edgetx icon indicating copy to clipboard operation
edgetx copied to clipboard

chore(bw): tidy up Radio Setup menu

Open philmoz opened this issue 10 months ago • 5 comments

Summary of changes:

  • Indent the 'Mode' label in the 'Sound' section.
  • Move the 'Contrast' setting to be directly under the 'Brightness' setting in the 'Backlight section'.
  • Indent the 'Contrast setting.

philmoz avatar Apr 29 '24 01:04 philmoz

Very nice!

inventor7777 avatar Apr 29 '24 21:04 inventor7777

I think we should move away from the indent been part of the translation string and use INDENT_WIDTH or similar in lcdDrawText() instead. It helps a lot when you have to resue that word

3djc avatar Apr 30 '24 06:04 3djc

I think we should move away from the indent been part of the translation string and use INDENT_WIDTH or similar in lcdDrawText() instead. It helps a lot when you have to resue that word

I've cleaned up the usage of the INDENT macro in the code and replaced it with INDENT_WIDTH. Cleaning up all the translation strings is a big job and probably something for another PR.

philmoz avatar May 02 '24 07:05 philmoz

Clearly, one step at a time!

3djc avatar May 02 '24 07:05 3djc

GRAPHICS was another 9x legacy define... so good riddance!

pfeerick avatar May 02 '24 07:05 pfeerick