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

📝 [WIP] Clarify TJC display support #1105

Draft
wants to merge 5 commits into
base: import-2.1.x
Choose a base branch
from

Conversation

thisiskeithb
Copy link
Member

@thisiskeithb thisiskeithb commented Sep 29, 2024

Description

  • TJC displays are mostly unsupported due to missing graphics
  • Copy Ender-3 S1 LCD Files README to Ender-3 V2 Neo folder

TJC display (ie what you have) uses TJC_SET/tjc.tft. We cannot generate those files, so there is no screen updates for this display provided by Marlin.

Originally posted by @ellensp in MarlinFirmware/Marlin#27446 (comment)


Benefits

Improves documentation for TJC displays

Related Issues

@queeup
Copy link

queeup commented Oct 4, 2024

@thisiskeithb
Copy link
Member Author

Maybe @mriscos can help with this support?

Marlin’s Ender-3 V2 LCD support / Pro UI has diverged quite a bit from @mriscoc’s fork, but if they are willing to patch things up, we’d appreciate the help.

@mriscoc
Copy link

mriscoc commented Oct 4, 2024

For TJC compatibility the only option is to use the iconset 9
So change the line:
#define ICON 7 // 7.ICO
to

#define ICON 9 // 9.ICO

Sometime ago there was a way to select the iconset by defining:

#define USE_STOCK_DWIN_SET

But it is not present in current Marlin version. I think that some additional icons were added which are incompatible with the 9.ICO. But maybe the standard CrealityUI keeps the compatibility.

@ellensp
Copy link
Contributor

ellensp commented Oct 4, 2024

USE_STOCK_DWIN_SET was removed in
commit MarlinFirmware/Marlin@0f09554
Perhaps it needs to be brought back.

@thinkyhead
Copy link
Member

When icon set 9 is selected do all the icons appear and none are missing with the UIs that support the TJC screen? If so, then it sounds like whenever a TJC screen is selected then ICON should automatically be set to 9. And, if this means that a TJC screen cannot support MarlinUI because it needs more icons, then we can also add a sanity check for that, or if possible check for the alternative icon set and display different icons for MarlinUI in that case.

@thisiskeithb
Copy link
Member Author

I've added 76fc9c0 & a8a75d6 Ender-3 V2 NEO README changes to the Ender-3 S1 README in 029db09.

I tried to make it clear that the Ender-3 V2 NEO README was copied from the Ender-3 S1 README in the commit title for b6d650d: "Copy Ender-3 S1 LCD Files README to Ender-3 V2 Neo folder" as well as the PR description, but let me know if there's a better way to highlight this in the future.

@thisiskeithb
Copy link
Member Author

thisiskeithb commented Oct 6, 2024

When icon set 9 is selected do all the icons appear and none are missing with the UIs that support the TJC screen?

I have a genuine DWIN screen, so I can't test this.


@albka1986: See @thinkyhead's comment above.

Using the latest bugfix-2.1.x, can you try changing:

#define ICON 7 // 7.ICO

to:

#define ICON 9 // 9.ICO

in dwin_set.h?

@thisiskeithb thisiskeithb marked this pull request as draft October 6, 2024 17:50
@thisiskeithb thisiskeithb changed the title 📝 Clarify TJC display support 📝 [WIP] Clarify TJC display support Oct 6, 2024
@thisiskeithb thisiskeithb added the Don't Merge The PR is not complete yet label Oct 6, 2024
@albka1986
Copy link

I updated the line, but still see the issue with displaying info:
image
image

@ellensp
Copy link
Contributor

ellensp commented Oct 7, 2024

the original code moves the text aroud also

Eg

void ICON_Control() {
  constexpr frame_rect_t ico = { 17, 226, 110, 100 };
  constexpr text_info_t txt = { 61, { 405, TERN(USE_STOCK_DWIN_SET, 446, 447) }, 27, 15 };
  ICON_Button(select_page.now == PAGE_CONTROL, ICON_Control_0, ico, txt, GET_TEXT_F(MSG_CONTROL));
}

So It needs more than just #define ICON 9

@albka1986
Copy link

What should I change? Could you please clarify?

@thisiskeithb thisiskeithb force-pushed the pr/clarify_tjc_display_support branch from e192f73 to c22761e Compare January 15, 2025 01:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs / README Don't Merge The PR is not complete yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants