File list issue - Githubissues

DaGr70 / Marlin_Longer_LK4_pro

Optimized firmware for RepRap 3D printers based on the Arduino platform. Modified with a new DWIN T5UID1 touchscreen implementation.

GNU General Public License v3.0

17 stars 1 forks source link

File list issue #2

Open thghca opened 3 years ago

thghca commented 3 years ago

Hi. I using yours screen firmware with mvr fork (https://github.com/mrv96/Marlin2-LK4-Pro). I tried yours and bug still here: When i open sd menu it displays file list but ui not react touches. Bug occurs only if less then 3 files on sd present, if more it works fine. Back button not react too. Only way exit - remove sd. Subfolders behave same exept level up button works.

As workaround for now i use some placeholder files.

DaGr70 commented 3 years ago

Hi, In fact, I meet similar issues with my Sdcard and my irmware (forked from [Desuuuu/Marlin]https://github.com/Desuuuu/Marlin) :

if number of files is under 3 : the "selection" button don't react. (same issue when located into folders)
if exist a folder name with lenght upper than 8 characters : the firmware don't reconize concerned folder or another folder and buttons don't work properly
If folder name contain "space" character => similar issue

I don't meet this issue previously because my SD card was provided by Longer precharged with some examples.

I propose to pull these issues to original branch to work together to solve them

Thanks Thghca.

thghca commented 3 years ago

I connected speaker to screen. It produces weird chirping when buttons touched. Only "Open" button chirps, but do nothing because nothing selected.

thghca commented 3 years ago

Back button not working because dgus_display.DisableControl makes no difference between back and file1 buttons.

thghca commented 3 years ago

Quickfix is comment whole DGUSTxHandler::SetFileControlState body. I don't see much reason in disabling controls anyway.

DaGr70 commented 3 years ago

I don't have any speaker, but into original Desuuuu screen, Firmware, Desuuu provided 3 *.wav file into DWIN_SET. Them shoud be used. I'll update screen firmware with these files.

Concerning DGUSTxHandle , Thanks for your investigation. I'll try your proposal.

DaGr70 commented 3 years ago

Hi thghca, Desuuuu provided me some information: index of control should be in accordance with "DGUS_Control.h"

So I'll try to modify touchscreen firmware to take the same Index Control than used by original Reloaded firmware Capture

DaGr70 commented 3 years ago

Hi thghca,

I've updated my firmware following Desuuuu advice. It works with less than 3 files. it stay an issue with the name of folder (must be < 8 character and witout space)

New firmwares and sources available here