edgetx icon indicating copy to clipboard operation
edgetx copied to clipboard

Custom global warning

Open piotrva opened this issue 2 years ago • 26 comments

Fixes #1276

Summary of changes:

  • Added ability to display custom global warning, that is displayed on startup and each model load
  • The warning has user-defined title and text
  • Added menu entries to enable/disable feature as well as define title
  • Text only to be changed by YAML file edit or using Companion as it is to long to enter by radio GUI

image

Comments: Together with model notes / checklist it allows to display critical warnings globally and per model.

Notes:

  • All changes were tested only on Radiomaster TX12
  • Tested that opening yaml file without positions for the settings is safe (defaults to this option disabled - backwards compatible)

piotrva avatar Jan 13 '22 03:01 piotrva

Coool!

Eldenroot avatar Jun 29 '22 15:06 Eldenroot

So there will not be any way how to set it via rx, not companion? Maybe some txt file with text (like note feature) and load it for specified warning? Txt file could be just copied to sd card on pc so no companion needed. I am just asking, this is a really cool idea and feature, I hope it will be merged into 2.8

Eldenroot avatar Aug 20 '22 05:08 Eldenroot

@Eldenroot To keep it flexible and simple simultaneously I decided to:

  • have warning title (short text) with ability to be edited from radio
  • have warning text to be edited from companion / YAML file.

As YAML file is only a specially formatted text file it might be edited without companion with no effort. In my opinion there is no need to make a mess with another file just to store a line or two of text

piotrva avatar Aug 26 '22 12:08 piotrva

@piotrva - OK, sounds good!

Eldenroot avatar Aug 27 '22 09:08 Eldenroot

DA translation add radio/translations/da.h :-)

#define TR_CUSTOM_ALARM_TITLE "Bruger titel" #define TR_CUSTOM_ALARM "Bruger"

HThuren avatar Nov 04 '22 21:11 HThuren

CZ translation add radio/translations/cz.h

#define TR_CUSTOM_ALARM_TITLE "Vlastní název" #define TR_CUSTOM_ALARM "Vlastní"

Eldenroot avatar Nov 05 '22 09:11 Eldenroot

Translators please could you check? :) @ParkerEde @Eldenroot - OK @ulfhedlund @robustini @zyren @ajjjjjjjj @ToshihiroMakuuchi @HThuren - OK

Eldenroot avatar Jan 01 '23 09:01 Eldenroot

IT

#define TR_CUSTOM_ALARM_TITLE "Titolo utente" #define TR_CUSTOM_ALARM "Utente"

robustini avatar Jan 01 '23 11:01 robustini

PL

#define TR_CUSTOM_ALARM_TITLE          "Własny tytuł"
#define TR_CUSTOM_ALARM                "Własny"

ajjjjjjjj avatar Jan 01 '23 12:01 ajjjjjjjj


CN #define TR_CUSTOM_ALARM_TITLE "自定义标题" #define TR_CUSTOM_ALARM "自定义"



TW #define TR_CUSTOM_ALARM_TITLE "自定義標題" #define TR_CUSTOM_ALARM "自定義"


zyren avatar Jan 01 '23 14:01 zyren

DE

#define TR_CUSTOM_ALARM_TITLE "Benutzer-Titel" #define TR_CUSTOM_ALARM "Benutzer"

ParkerEde avatar Jan 01 '23 15:01 ParkerEde

JP

#define TR_CUSTOM_ALARM_TITLE "カスタムタイトル" #define TR_CUSTOM_ALARM "カスタム"

ToshihiroMakuuchi avatar Jan 02 '23 06:01 ToshihiroMakuuchi

SE

#define TR_CUSTOM_ALARM_TITLE "Anpassad rubrik" #define TR_CUSTOM_ALARM "Anpassat meddelande"

ulfhedlund avatar Jan 02 '23 16:01 ulfhedlund

Oh, I see that the milestone is again postponed... :( is there anything what can I do to make it integrated into 2.10?

Eldenroot avatar Apr 16 '23 09:04 Eldenroot

Oh, I see that the milestone is again postponed... :( is there anything what can I do to make it integrated into 2.10?

Making a rebase would help a lot

gagarinlg avatar May 05 '23 14:05 gagarinlg

Oh, I see that the milestone is again postponed... :( is there anything what can I do to make it integrated into 2.10?

Making a rebase would help a lot

Above my coding knowledge together with LVGL :( srry

Eldenroot avatar May 07 '23 14:05 Eldenroot

rebased, but probably not functional right now, at least companion is probably broken

gagarinlg avatar May 07 '23 19:05 gagarinlg

Is this still relevant or has the interactive checklist superseded it?

philmoz avatar Nov 01 '23 06:11 philmoz

This is not directly related to checklists, is a standalone feature so still relevant.

pfeerick avatar Nov 01 '23 08:11 pfeerick

Oh, no... again postponed :(

Eldenroot avatar Dec 12 '23 19:12 Eldenroot

Both this and another PR will be revisited soon once 2.10 is branched off. Just don't have the time or energy to fight with the rebase for them.

pfeerick avatar Dec 12 '23 21:12 pfeerick