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

Update TalkBack to v14.1 #62

Merged
merged 0 commits into from
Apr 12, 2024

Conversation

PatrykMis
Copy link

Like @thestinger have asked in #50 (comment).

Tested and updated everything. In addition, two small upstream issues have been fixed.

@PatrykMis PatrykMis changed the title Updates and cleanup Update TalkBack to v14.1 Mar 11, 2024
@PatrykMis
Copy link
Author

Rebased onto recently pushed v14.1 release.

@thestinger
Copy link
Member

Thanks, we'll try to get to this soon.

@PatrykMis PatrykMis marked this pull request as draft March 12, 2024 02:21
@PatrykMis
Copy link
Author

I have to fix few upstream crashes though, especially in TalkBack's preferences.

@PatrykMis PatrykMis marked this pull request as ready for review March 12, 2024 03:37
@PatrykMis
Copy link
Author

Crashes fixed, one thing which seams to not work at all is haptic feedback, I really don't have an idea how to debug and restore it.

@thestinger
Copy link
Member

Missing a permission or something?

@thestinger
Copy link
Member

I don't understand why they do such a bad job with this.

@PatrykMis
Copy link
Author

Ok, haptic feedback is not working properly on some devices even on version from Play Store, so I think it's not worth fixing.

Icon detection is not working on FOSS version because it needs proprietary module, so considering to remove this option from preferences, what do you think?

Also, to remove Network permission? In fact, it's not being used for anything other than analytics and performance metrics, OCR is being performed by MLKit offline.

@thestinger
Copy link
Member

Ok, haptic feedback is not working properly on some devices even on version from Play Store, so I think it's not worth fixing.

That seems unfortunate since they'll probably fix it there.

@thestinger
Copy link
Member

Maybe it's just broken on older Android versions? Where is it broken?

@thestinger
Copy link
Member

Also, to remove Network permission? In fact, it's not being used for anything other than analytics and performance metrics, OCR is being performed by MLKit offline.

Seems fine to remove it.

@PatrykMis
Copy link
Author

PatrykMis commented Mar 12, 2024

That seems unfortunate since they'll probably fix it there.

If they fix it, we can try with decompiling Play variant, as they are not obfuscating source code at all; also no specific Proguard optimizations there.

@PatrykMis
Copy link
Author

Maybe it's just broken on older Android versions? Where is it broken?

Rather on A14, I will test it on other devices than Pixel 8 tomorrow.

@PatrykMis PatrykMis force-pushed the gos-cleanup-231206 branch 5 times, most recently from 98384cd to e699a1d Compare March 15, 2024 03:47
@PatrykMis PatrykMis force-pushed the gos-cleanup-231206 branch 2 times, most recently from cf4570d to 0d435f6 Compare March 23, 2024 06:17
@thestinger
Copy link
Member

@PatrykMis Do you think this should be merged? It's looking like there was a lot broken.

@PatrykMis
Copy link
Author

PatrykMis commented Mar 24, 2024

In general, it is ready to merge: lots of new features have been added, including text editing in braille keyboard, etc. It has been tested extensively by me for half a month. Additionally, the haptic feedback is working well when touch feedback is enabled in system settings, too.

However, there are still some minor tweaks that can be made, such as removing the option for the stub function to download dynamic features for icon detection, as this will not work in the open-source version.

@PatrykMis PatrykMis force-pushed the gos-cleanup-231206 branch 3 times, most recently from ec6af73 to 9ab24b9 Compare March 25, 2024 09:37
@PatrykMis
Copy link
Author

PatrykMis commented Mar 25, 2024

@thestinger Now everything is working as expected, ready to merge (rebase).

@PatrykMis PatrykMis force-pushed the gos-cleanup-231206 branch 2 times, most recently from dfd0bff to 0ce79fe Compare April 10, 2024 08:53
@PatrykMis
Copy link
Author

PatrykMis commented Apr 10, 2024

Rebased with AGP updated to v8.3.2 and cleaner GIT commit history, no special changes.

@thestinger
Copy link
Member

@PatrykMis Going to try to include it in the next release.

@thestinger
Copy link
Member

@PatrykMis It would be nice to switch it to Java 17 since it's the last thing still using Java 11. Would need to be tested with that before shipping it that way though.

@PatrykMis PatrykMis deleted the gos-cleanup-231206 branch April 16, 2024 06:09
@thestinger
Copy link
Member

@PatrykMis Can you test the build we have at https://github.com/GrapheneOS/platform_external_talkback? Note: it's not signed, since we currently let it get signed with the OS releasekey but we could stop doing that to start updating it out-of-band at some point. It gets updated so infrequently that we haven't bothered though.

@matchboxbananasynergy
Copy link

I think being able to update it out-of-band can be very useful in the off chance some regression is introduced. It's best to not require an OS update for that.

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