Skip to content
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

curentPageID, lastPageID #7

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

InnuendoPi
Copy link

No description provided.

@sstaub
Copy link
Owner

sstaub commented Dec 8, 2022

I'm a little confused because you use return code 0x24 (buffer overflow) instead of 0x66

@InnuendoPi
Copy link
Author

I've checked this document. 0x24 is not in use
https://wiki.iteadstudio.com/Nextion_Instruction_Set#Format_of_Device_Return_Data
As a preinit event 0x24 returns inputString[1] as currentPage. Event rise up only once, when you change the page. So this trigger makes it easy to keep track of currentPage and lastPage. Sendme event (0x66) returns inputString[1] as "active" page. In my tests sendme returns last active page ID while in preinit state.
Gruß aus Köln.

@sstaub
Copy link
Owner

sstaub commented Dec 8, 2022

0x24 wird benutzt, siehe Screenshot auf der Webseite. Ich habe in meiner Codeerweiterung leider einen größeren Bug eingefügt, so das etwas dauern wird bis ich den Fehler gefunden habe.
Bildschirm­foto 2022-12-08 um 11 25 30

@InnuendoPi
Copy link
Author

Dann ist 0x24 keine geeignete Wahl. Kann ich Dich irgendwie bei der Fehlersuche unterstützen?

@sstaub
Copy link
Owner

sstaub commented Dec 8, 2022

Ich muss den Code unter Umständen neu aufsetzen, die Erweiterungen arbeiten mit aufwendigen Callbacks, ich denke da liegt das Problem. Es gibt dann die Möglichkeit eine Callback Funktion aufzurufen wenn es eine Änderung der Seite gibt. Der Code wird dann über den Returncode 0x66 funktionieren. Du must dann die sendme Befehle dementsprechend bei jeder Seite einfügen. Hat im Simulator problemlos funktioniert.

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