Skip to content

Overlay issue if other pwnagotchi in range #3

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

Open
mkoterski opened this issue Feb 8, 2021 · 3 comments
Open

Overlay issue if other pwnagotchi in range #3

mkoterski opened this issue Feb 8, 2021 · 3 comments
Assignees
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed

Comments

@mkoterski
Copy link
Contributor

mkoterski commented Feb 8, 2021

If there is another pwnagotchi in range, the notification will overlap with what is being displayed by display-password.py.

image

According to the documentation there are the two callbacks that might be of help here

on_peer_detected - Called when a new peer is detected.

and

on_peer_lost - Called when a known peer is lost.

Not sure if I am skilled enough but I might have a look into this if there's interest/time.

@c-nagy
Copy link
Owner

c-nagy commented Feb 9, 2021

Good catch! I think this can be fixed in a pretty straightforward way. I just wish I had another Pi to test peering and everything with.

If you can put in a fix that would be greatly appreciated. I don't know if the ideal approach would be to have the peer status and cracked passwords alternate every N screen refreshes, or if one should just override the other?

@c-nagy c-nagy added help wanted Extra attention is needed bug Something isn't working labels Feb 9, 2021
@c-nagy c-nagy added enhancement New feature or request good first issue Good for newcomers and removed enhancement New feature or request labels Feb 9, 2021
@NBalciunas
Copy link

Sorry first time when I actually know how to fix a problem so I don't know how to push my fix into here, but I forked this plugin and made it so that you set the coordinates yourself instead of it detecting your screen. It makes it that you can modify it to be a bit higher or to the side so the texts don't collide.

@NBalciunas
Copy link

Example image with the exp plugin
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants