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

Waveshare_3 correct position on the screen #39

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

Conversation

drpurr1975
Copy link

No description provided.

@@ -39,7 +39,7 @@ def on_ui_setup(self, ui):
memenable = True
logging.info(
"Pwnagotchi Clock Plugin: memtemp is enabled")
if ui.is_waveshare_v2():
if ui.is_waveshare_v2() or ui.is_waveshare_v3():
pos = (130, 80) if memenable else (200, 80)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
pos = (130, 80) if memenable else (200, 80)
pos = (110, 80) if memenable else (200, 80)

You will also need to change this to 110 otherwise if memtemp is enabled it will obscure the clock trough i can't say what change this will bring to users of waveshare v2 because presumatly this worked fine on said versions so maybe it would make more sense to add another check for v2/v3 to differentiate between those two values

@jLynx
Copy link

jLynx commented May 10, 2023

No point commenting. This entire repo has been abandoned

@MNarath1
Copy link

No point commenting. This entire repo has been abandoned

you are probably right trough i figured even if this doesn't help much if someone else looks for a fix for said issue and they come across this PR i might as well document some small changes for them that are also needed (or for myself when i inevadably forget about it and then look the same PR up again)

@jLynx
Copy link

jLynx commented May 10, 2023

Yeah fair enough. Was just a heads up anyway

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.

3 participants