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

Remove the vibration while charging #1768

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

minacode
Copy link
Contributor

@minacode minacode commented Jun 5, 2023

This fixes #724 where the vibration interupts the physical connection with the charger and therefore triggers a new charging event and vibration, ending in a cycle of vibrations while charging.

This fixes a bug where the vibration interupts the physical connection with the charger and therefore triggers a new charging event and vibration, ending in a cycle of vibrations while charging.
@minacode minacode changed the title DisplayApp.cpp: Remove the vibration from OnChargingEvent Remove the vibration from OnChargingEvent Jun 5, 2023
@minacode minacode changed the title Remove the vibration from OnChargingEvent Remove the vibration while charging Jun 5, 2023
@github-actions
Copy link

github-actions bot commented Jun 5, 2023

Build size and comparison to main:

Section Size Difference
text 374576B -32B
data 948B 0B
bss 63504B 0B

@Itai-Nelken
Copy link
Contributor

I like this, I have experienced this problem many times. But I would still like some way to be sure the watch is charging other than the tiny icon that is hard to see. Maybe the battery settings page opens when charging?

@minacode
Copy link
Contributor Author

I like this, I have experienced this problem many times.

Nice :) It seems to be a common issue.

But I would still like some way to be sure the watch is charging other than the tiny icon that is hard to see. Maybe the battery settings page opens when charging?

I think that would belong to another PR. This one is only about removing the bug.

@everypizza1
Copy link
Contributor

That's not a problem I have had on the sealed watch, but the vibration is very loud.

@mark9064
Copy link
Member

Just FYI there is also #1876

@minacode
Copy link
Contributor Author

Which I like even more! 😄 I hope your gets merged. We can close this one if it did.

@LinuxinaBit
Copy link

I do like the tactility though, what about only doing the vibration if there hasn’t been a charging event in the last ~1 minute?

@minacode
Copy link
Contributor Author

I will keep this PR as simple as possible and just remove the one line.

Copy link
Member

@mark9064 mark9064 left a comment

Choose a reason for hiding this comment

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

I think it also makes sense to remove the charging event from displayapp as apps pull charging state directly from the batterycontroller; after this change the message would be unused.

@minacode
Copy link
Contributor Author

Totally. I removed it.

@mark9064 mark9064 added this to the 1.16.0 milestone Nov 24, 2024
@mark9064 mark9064 added the enhancement Enhancement to an existing app/feature label Nov 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement to an existing app/feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Constant vibration when charging
5 participants