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

Support color depth for 8 colors screen #21

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

Conversation

eggfly
Copy link

@eggfly eggfly commented Feb 17, 2023

Replace LS013B7DH06 from Breakout Board and tested.
It needs tripled RAM size than mono screen.

I added support for color depth because sharp sells a memory LCD with a color depth of 3. I have tested this library on that LCD with a feather m0 and it works. The part number for this LCD is LS013B7DH06 and is compatible with existing sharp memory LCD breakout boards.

@eggfly eggfly changed the title Support color depth for 8 color screen Support color depth for 8 colors screen Feb 17, 2023
@eggfly eggfly force-pushed the master branch 2 times, most recently from 8ddfa25 to 3f5aac5 Compare February 17, 2023 10:55
@eggfly
Copy link
Author

eggfly commented Feb 17, 2023

All check passed, please take some time to do a review :)
@dhiltonp @PaintYourDragon @tdicola 😄

@eggfly
Copy link
Author

eggfly commented Mar 22, 2023

@ladyada Hi! Can you assign any team member to review this PR? Thanks!

@ladyada
Copy link
Member

ladyada commented Mar 22, 2023

hi please start by not changing the object instantiation argument order - thanks! :)

@eggfly eggfly force-pushed the master branch 2 times, most recently from 9d9c7fc to f988367 Compare March 22, 2023 04:01
Replace LS013B7DH06 from Breakout Board and tested.
It needs tripled RAM size than mono screen.
@eggfly
Copy link
Author

eggfly commented Mar 22, 2023

@ladyada Modified argument order, done!

@eggfly
Copy link
Author

eggfly commented Jul 7, 2023

Ping! Adafruit does not has a product that uses LS013B7DH06 3 bits color screen, so this PR can't be merge? Is that the reason?

@ladyada
Copy link
Member

ladyada commented Jul 8, 2023

no, we dont have one. do you have a monochrome display you can verify it still works?

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