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

Add Compatibility with ST7789 LCD (T-Display S3) #1811

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

JoKenPo
Copy link

@JoKenPo JoKenPo commented Aug 17, 2024

This pull request introduces changes to make the code compatible with the ST7789 display used in the T-Display S3 board. This is my first time developing with Arduino, so I’m aware that the code is far from perfect.

The primary motivation for this PR is to open up discussions and gather feedback on how we can improve this implementation. I believe that adding compatibility with the T-Display S3 could significantly increase community engagement, given the popularity of this board.

I'm looking forward to your feedback and suggestions. Thank you for your attention!

@revoxhere
Copy link
Owner

That's a great addition! Can you share a photo of how it looks like right now?

@JoKenPo
Copy link
Author

JoKenPo commented Aug 17, 2024

I was only able to convert this screen and it still flickers a little.
tdisplay-s3

As I said, this is really my first contact with Arduino, what I tried to do was convert what is done with the Adafruit_LiquidCrystal lib to TFT_eSPI.h.

That's why I published this PR to have a review.

I don't even know about good practices in Arduino :p

@revoxhere
Copy link
Owner

It's a good start, much more can be done with a color screen however. I have one on hand with this controller so I'll probably try to add a fancier UI to your baseline in the future

@revoxhere revoxhere self-assigned this Aug 18, 2024
@revoxhere revoxhere added enhancement New feature or request in progress This is currently being worked on labels Aug 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request in progress This is currently being worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants