Skip to content

Conversation

@Xaositek
Copy link
Contributor

@Xaositek Xaositek commented Jul 14, 2025

  • Message frame has default options for new messages
  • Clock updates for Chatter 2 and TDeck
  • Remove Brightness Settings for HAS_TFT flag

@Xaositek Xaositek changed the title Clock refinements Clock, TDeck and Brightness Refinements Jul 14, 2025
@Xaositek Xaositek marked this pull request as ready for review July 14, 2025 23:35
@Xaositek Xaositek changed the title Clock, TDeck and Brightness Refinements Message frame New Message Options and Clock / TDeck / Brightness Refinements Jul 15, 2025
@thebentern thebentern requested a review from Copilot July 15, 2025 01:55
Copy link
Contributor

Copilot AI left a 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 textMessageBaseMenu and route navigation when no incoming message exists
  • Update ClockRenderer to support CHATER_2 scaling and T_DECK y-offset, and remove an unused variable
  • Revise brightness support flags in screenOptionsMenu and 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_found is true or false.
void menuHandler::textMessageBaseMenu()

src/graphics/draw/ClockRenderer.cpp:290

  • Double-check that this added #endif matches the correct #ifdef (SENSECAP_INDICATOR) and doesn’t prematurely close another block, which could lead to missing code under certain builds.
#endif

@thebentern thebentern merged commit 625a529 into master Jul 15, 2025
52 checks passed
@Xaositek Xaositek deleted the clock_refinements branch July 15, 2025 12:42
oscgonfer pushed a commit to fablabbcn/smartcitizen-meshtastic that referenced this pull request Jul 21, 2025
…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>
oscgonfer pushed a commit to fablabbcn/smartcitizen-meshtastic that referenced this pull request Jul 22, 2025
…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>
oscgonfer pushed a commit to fablabbcn/smartcitizen-meshtastic that referenced this pull request Jul 23, 2025
…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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants