Improved Status Area for DWIN Display

[Jyers]

Description

An improved status area for the Ender 3 V2 DWIN LCD display.

This is a small portion of a complete rewrite I have done for the DWIN display, which is why it contains a couple new global variables which will come into use as I submit more aspects of the rewrite in future PRs.

Requirements

An Ender 3 V2 with its stock DWIN Display

Benefits

Significantly more information in the status area of the display. Including flow rate, fan speed, and the current x, y, and z axis coordinates.

Configurations

Ender 3 V2 Config.zip

[ETE-Design]

@Jyers Mabye you could look at #19371 and make a complere rewrite to make if work perfect with Marlin features?

[Jyers]

@ETE-Design What I have already written should be able to be adapted to tie into the marlin UI features, so as time goes on I will most likely slowly migrate the features to the marlin UI framework.

[CRCinAU]

Other than the two comments - I have no other objections to this code. I have tested and it seems to do exactly what it states - which is a good addition to the DWIN ecosystem.

[CRCinAU]

On the downwards part of a G29 probe, I noticed that the Z value hit 726.4. I'm not exactly sure why this is the case. Someone might be able to enlighten us here.

[CRCinAU]

@thinkyhead - I'm trying to apply this to the current bugfix to test the changes, but it seems things are a bit confused. I normally grab a patch of the PR by adding .patch to the end of the URL, which gives me:
https://github.com/MarlinFirmware/Marlin/pull/20983.patch

When I apply this patch, a ton of it fails.

 $ patch -p1 < ../../20983.patch 
patching file Marlin/src/lcd/dwin/e3v2/dwin.cpp
Hunk #1 succeeded at 203 (offset -4 lines).
Hunk #2 FAILED at 1535.
Hunk #3 FAILED at 1580.
Hunk #4 FAILED at 1803.
3 out of 4 hunks FAILED -- saving rejects to file Marlin/src/lcd/dwin/e3v2/dwin.cpp.rej
patching file Marlin/src/lcd/dwin/e3v2/dwin.cpp
Hunk #1 FAILED at 1593.
Hunk #2 FAILED at 1632.
Hunk #3 FAILED at 1874.
3 out of 3 hunks FAILED -- saving rejects to file Marlin/src/lcd/dwin/e3v2/dwin.cpp.rej
patching file Marlin/src/lcd/dwin/e3v2/dwin.cpp
Hunk #2 succeeded at 1541 (offset 23 lines).
Hunk #3 FAILED at 1590.
Hunk #4 FAILED at 1859.
2 out of 4 hunks FAILED -- saving rejects to file Marlin/src/lcd/dwin/e3v2/dwin.cpp.rej

Do you have any suggestions on how to fix this? I can't say I've had this problem before - but maybe from additional commits that were then shifted elsewhere?

[thinkyhead]

I'm trying to apply this to the current bugfix to test the changes

I generally just use git plus GitHub Desktop when I want to compare things. So first I might just look at the differences between this PR and the current bugfix code…

git fetch upstream
git checkout upstream/bugfix-2.0.x -b changes_from_20983
git diff HEAD pr/20983 | git apply

Anyway, all you really need to do is check out the PR directly, because it is already applied to the current bugfix-2.0.x. If you wanted to bring that up to date after more commits are added to bugfix, you can just use git merge upstream/bugfix-2.0.x.

If you have the branch checked out and all you want to do is see the differences in isolation, or squashed to a single commit, you could use…

git checkout pr/20983 ;# as checked out by Github Desktop
git fetch upstream
git merge upstream/bugfix-2.0.x ;# latest commits
git reset upstream/bugfix-2.0.x ;# differences in isolation

[CRCinAU]

@thinkyhead - Thanks I'm pretty sure I got this right. So from a UI perspective, the only thing I see that doesn't work atm is the ???.? and blink parts.

Is that expected with the current status?

[mriscoc]

Thank you for your PR I applied it successfully to my fork.

[thinkyhead]

the only thing I see that doesn't work

The logic to blink the positions is similar to that used for other displays. If it's not working now then it can be adjusted. I need to flash and test some other things on the Ender-3 V2 so I will test these updates shortly.

In the long run, of course, we want to get rid of the custom menu code in dwin.cpp and replace it with MarlinUI. The PR for that is almost working over at #19371.