-
-
Notifications
You must be signed in to change notification settings - Fork 2k
Message frame New Message Options and Clock / TDeck / Brightness Refinements #7344
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces a new base menu for composing messages, refines clock display behavior for Chatter 2 and TDeck, and adjusts brightness and navigation logic across menus.
- Add
textMessageBaseMenuand route navigation when no incoming message exists - Update
ClockRendererto support CHATER_2 scaling and T_DECK y-offset, and remove an unused variable - Revise brightness support flags in
screenOptionsMenuand include T_DECK in TFT color picker - Shift header icon positioning and improve text message menu flow in
Screen::handleInputEvent
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/graphics/draw/MenuHandler.h | Declared new textMessageBaseMenu prototype |
| src/graphics/draw/MenuHandler.cpp | Implemented textMessageBaseMenu and updated menu flags |
| src/graphics/draw/ClockRenderer.cpp | Added scale and yOffset adjustments for CHATER_2 and T_DECK; removed unused line variable |
| src/graphics/SharedUIDisplay.cpp | Adjusted iconRightEdge calculation from -1 to -2 |
| src/graphics/Screen.cpp | Updated input handling to show base or response message menu |
Comments suppressed due to low confidence (3)
src/graphics/draw/MenuHandler.cpp:349
- [nitpick] The enum name 'optionsNumbers' and member 'enumEnd' are ambiguous. Consider renaming to something like 'TextMessageOption' and 'OptionCount' for clearer intent.
enum optionsNumbers { Back, Preset, Freetext, enumEnd };
src/graphics/draw/MenuHandler.cpp:347
- [nitpick] This new menu introduces user-facing options and a lambda callback. Consider adding unit or integration tests to validate both the Preset and Freetext branches and when
kb_foundis true or false.
void menuHandler::textMessageBaseMenu()
src/graphics/draw/ClockRenderer.cpp:290
- Double-check that this added
#endifmatches the correct#ifdef(SENSECAP_INDICATOR) and doesn’t prematurely close another block, which could lead to missing code under certain builds.
#endif
…nements (meshtastic#7344) * Clock updates and some TDeck corrections * TDeck Brightness Works in TFT Builds * Remove HAS_TFT from enabling Brightness, disable Brightness for TDeck * Add default textMessage frame actions and adjust SharedUIDisplay --------- Co-authored-by: Ben Meadors <benmmeadors@gmail.com>
…nements (meshtastic#7344) * Clock updates and some TDeck corrections * TDeck Brightness Works in TFT Builds * Remove HAS_TFT from enabling Brightness, disable Brightness for TDeck * Add default textMessage frame actions and adjust SharedUIDisplay --------- Co-authored-by: Ben Meadors <benmmeadors@gmail.com>
…nements (meshtastic#7344) * Clock updates and some TDeck corrections * TDeck Brightness Works in TFT Builds * Remove HAS_TFT from enabling Brightness, disable Brightness for TDeck * Add default textMessage frame actions and adjust SharedUIDisplay --------- Co-authored-by: Ben Meadors <benmmeadors@gmail.com>
Uh oh!
There was an error while loading. Please reload this page.