search

pilot51 / voicenotify

Android app that speaks notifications
https://voicenotify.app
Apache License 2.0
152 stars 54 forks source link

the current desktop/apps page is reported, although it shouldn't #23

Closed pldimitrov closed 6 years ago

pldimitrov commented 10 years ago

I unticked the following apps:

Active applications Android System com.samsung.InputEventApp System UI SystemInfo SASlideShow Samsung Apps

And only ticked the device states:

Screen Off Headset On

The app reports the current desktop/apps page "Page X of Y" even though the screen is on and the headset is off.

Model number: GT-I9100 Android version: 4.1.2 Baseband version: I9100XXLS8 Kernel version: 3.0.31-899555 dpi@DELL228 #3 SMP PREEMPT Thu Jan 31 14:48:54 KST 2013 Build number: JZO54K.I9100XWLSD

pilot51 commented 10 years ago

This is an issue with most Samsung versions of Android and affects all accessibility services that use TTS, not just VN.

I haven't confirmed, but I believe the fix/workaround is to disable the Samsung TTS engine and use a different engine such as Google TTS.

I'll leave this issue open until I know you're satisfied with the solution.

pilot51 commented 6 years ago

I'm confident this was fixed in 682bbb7 by reading notifications through Notification Listener Service instead of Accessibility Service. As a side effect of this change, VN no longer supports versions of Android prior to 4.3. Of course, if the issue only existed prior to 4.3, this isn't really a fix and anyone still affected should upgrade.