Various bugfixes, timer rollover bugs, encoder, long press freeze bug, encoder issue at higher values #2967

rondlh · 2024-12-11T11:09:13Z

Various bugfixes and small enhancements

Requirements

No specific requirements

Description

Bug fixes
Solves various timer rollover bugs as discussed in #2832
Solves the encoder not working well bug at higher values bug (speed and feedrate)
Solves the long press freeze bug when pressing the title bar
Corrects the UART6 pins on MKS TFT35 V1.0

Enhancements
improved handling of back button and long press button, 2 additional serial baud rates
Long press functionality can now easily be added to any icon.
Adds long press functions (jump to high temp without clicking 20 times, adjust movement speed, jump to temp from extruder menu)
Allow to jump from the first page to the last page in the console and visa versa.

Benefits

Squashing bugs, and small enhancements improving usability. For example, TFTs that don't have a encoder need to be pressed 20 times to go to 200 degrees C. The long press functionality makes that a 1 click action now. The same is true for the bed, which can jump to 75 degrees, or any degree you set.

Related Issues

#2832 (closed, but was not solved).

…der bug Solves various timer rollover bugs Solves the encoder not working well bug Solves the long press freeze bug Corrects the UART6 pins on MKS TFT35 V1.0 Enhancements, improved handling of back button and long press button 2 additional serial baud rates Adds several long press functions (jump temp, adjust movement speed, jump to temp from extruder menu)

rondlh · 2024-12-11T11:16:00Z

Anybody, please download and test, try long pressing the movement amount in the "move menu", or the "+" or "-" in the "Heat menu". The long press freeze bug should be solved now, but I still found some freezes in the console, when the motherboard is being restarted, I have not yet located that bug.
In the terminal you can now jump from the first page to the last and the other way around.

@bigtreetech Could you please add the current status to the releases? The last release if from 2021 and is quite outdated.

UPDATE: The back button has some issues, still needs fixing

"if (a == true)" is the same as "if (a)" "if (a == false)" is the same as "if (!a)" Some small comment formatting fixes Small optimization

digant73 · 2024-12-16T10:25:14Z

TFT/src/User/API/menu.c

@@ -709,7 +710,6 @@ LISTITEMS * getCurListItems(void)
 GUI_POINT getIconStartPoint(int index)
 {
  GUI_POINT p = {curRect[index].x0, curRect[index].y0};
-


do not remove the empty line before any return statement. It has been added to highlight the return point

digant73 · 2024-12-16T10:28:31Z

TFT/src/User/API/menu.c

@@ -765,7 +765,6 @@ void menuRefreshListPage(void)
  for (uint8_t i = 0; i < ITEM_PER_PAGE; i++)
  {
    RAPID_PRINTING_COMM()  // perform backend printing loop between drawing icons to avoid printer idling
-


do not remove the empty line before any GUI call. It has been added to separate GUI from non GUI calls

digant73 · 2024-12-16T10:28:53Z

TFT/src/User/API/menu.c

@@ -799,7 +798,6 @@ void setMenu(MENU_TYPE menu_type, LABEL * title, uint16_t rectCount, const GUI_R
 void menuSetTitle(const LABEL * title)
 {
  curTitle = title;
-


as previous comment

digant73 · 2024-12-16T10:29:00Z

TFT/src/User/API/menu.c

@@ -820,7 +818,6 @@ void menuDrawTitle(void)
  if (toastRunning())
  {
    drawToast(true);
-


as previous comment

digant73 · 2024-12-16T10:29:09Z

TFT/src/User/API/menu.c

@@ -883,7 +880,6 @@ static void itemDrawIconPress(uint8_t position, uint8_t is_press)
    if (curListItems->items[position].icon == CHARICON_NULL)
    {
      GUI_ClearPrect(rect);
-


as previous comment

digant73 · 2024-12-16T11:03:50Z

TFT/src/User/API/Touch_Screen.c

@@ -174,6 +175,7 @@ static inline uint8_t TS_CalibrationEnsure(uint16_t x, uint16_t y)
    GUI_DispStringCenter(LCD_WIDTH / 2, LCD_HEIGHT - 40, (int32_t)LABEL_ADJUST_FAILED);
    GUI_DispDec(0, 0, lcd_x, 3, 0);
    GUI_DispDec(0, 20, lcd_y, 3, 0);
+    Buzzer_AddSound(100, 200);


move the buzzer call before the first GUI call and add an empty line before the first GUI call (separate GUI call from non GUI calls, when possible)

digant73 · 2024-12-16T11:06:00Z

TFT/src/User/API/Touch_Screen.c

-      while (TS_IsPressed() == false);
-
+      while (!TS_IsPressed());
+      Buzzer_AddSound(BUZZER_FREQUENCY_HZ, BUZZER_FREQUENCY_DURATION_MS);


empty line before this buzzer call. while, if, switch etc. must be always separated by empty line

digant73 · 2024-12-16T11:08:10Z

TFT/src/User/API/menu.c

@@ -4,6 +4,7 @@
 #include "Notification.h"

 #define STATUS_BAR_REFRESH_TIME 2000  // refresh time in ms
+uint8_t longPressed = 0;


empty line before this variable definition (separate by empty line #define etc. from variable definition etc.)

digant73 · 2024-12-16T11:09:40Z

TFT/src/User/API/menu.c

@@ -656,13 +658,12 @@ void drawBusySign(void)

    busySign.status = SYS_STATUS_BUSY;
  }
-


restore the empty line after any if, switch, while etc.

digant73 · 2024-12-16T11:14:22Z

TFT/src/User/API/menu.h

@@ -198,13 +199,11 @@ void showLiveInfo(uint8_t index, const LIVE_INFO * liveicon, bool redrawIcon);
 void displayExhibitHeader(const char * titleStr, const char * unitStr);
 void displayExhibitValue(const char * valueStr);

-KEY_VALUES menuKeyGetValue(void);
+KEY_VALUES menuKeyGetValue(bool returnLongPressed);

 // smart home
 #ifdef SMART_HOME


if the SMART_HOME is now always supported, remove this #ifdef from the entire project

@digant73 Thanks a lot for the feedback! I just pushed an update reflecting your input.

About the SMART_HOME thing, it's still selectable, my changes don't affect this.
But... I see no real reason why one would want to disable it, so it could be removed if you ask me.
If you agree, then I can take out the Smart Home option everywhere needed.

BTW: The current status should be made into a release... it's a stable usable base. The last release is from 2021, which is very outdated!

hhmm no, if still selectable, I would avoid to remove SMART_HOME (leave it as is)

…sistent As commented by digant73 on December 17th. Thanks! Moved "return" command to separate line to be more consistent.

Add more feedback, I missed a few that were folded together in the comment section

Some comments formatting, removed == true, == false

2febd02

"if (a == true)" is the same as "if (a)" "if (a == false)" is the same as "if (!a)" Some small comment formatting fixes Small optimization

rondlh marked this pull request as draft December 15, 2024 11:26

Menu fix, should not look to labels

3cf86f6

This comment was marked as duplicate.

Sign in to view

digant73 reviewed Dec 16, 2024

View reviewed changes

rondlh added 2 commits December 18, 2024 06:26

Restore some deleted blank lines, make use of "return" a bit more con…

ac38771

…sistent As commented by digant73 on December 17th. Thanks! Moved "return" command to separate line to be more consistent.

A few more lines restored and feedback processed

cb25aef

Add more feedback, I missed a few that were folded together in the comment section

bigtreetech marked this pull request as ready for review December 25, 2024 01:30

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Various bugfixes, timer rollover bugs, encoder, long press freeze bug, encoder issue at higher values #2967

Various bugfixes, timer rollover bugs, encoder, long press freeze bug, encoder issue at higher values #2967

rondlh commented Dec 11, 2024

rondlh commented Dec 11, 2024 •

edited

Loading

This comment was marked as duplicate.

digant73 Dec 16, 2024

digant73 Dec 16, 2024

digant73 Dec 16, 2024

digant73 Dec 16, 2024

digant73 Dec 16, 2024

digant73 Dec 16, 2024

digant73 Dec 16, 2024

digant73 Dec 16, 2024

digant73 Dec 16, 2024

digant73 Dec 16, 2024

rondlh Dec 17, 2024

digant73 Dec 18, 2024

		@@ -656,13 +658,12 @@ void drawBusySign(void)

		busySign.status = SYS_STATUS_BUSY;
		}

Various bugfixes, timer rollover bugs, encoder, long press freeze bug, encoder issue at higher values #2967

Are you sure you want to change the base?

Various bugfixes, timer rollover bugs, encoder, long press freeze bug, encoder issue at higher values #2967

Conversation

rondlh commented Dec 11, 2024

Requirements

Description

Benefits

Related Issues

rondlh commented Dec 11, 2024 • edited Loading

This comment was marked as duplicate.

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

rondlh commented Dec 11, 2024 •

edited

Loading