KModal appendToRoot changed to appendToOverlay

[nucleogenesis]

Summary

Previously changed from appendToBody to appendToRoot 671ccf1

@MisRob pointed out these should be appendToOverlay now instead.

References

Tests on #12651 should pass after this is merged and that dependabot branch is rebased.

Also is blocking #12571 to some degree so it should be rebased when this is merged.

Reviewer guidance

• Tests pass?

[github-actions]

Build Artifacts

Asset type	Download link
PEX file	kolibri-0.17.0a0.dev0_git.249.g62597caa.pex
Windows Installer (EXE)	kolibri-0.17.0a0.dev0+git.249.g62597caa-windows-setup-unsigned.exe
Debian Package	kolibri_0.17.0a0.dev0+git.249.g62597caa-0ubuntu1_all.deb
Mac Installer (DMG)	kolibri-0.17.0a0.dev0+git.249.g62597caa.dmg
Android Package (APK)	kolibri-0.17.0a0.dev0+git.249.g62597caa-0.1.4-debug.apk
TAR file	kolibri-0.17.0a0.dev0+git.249.g62597caa.tar.gz
WHL file	kolibri-0.17.0a0.dev0+git.249.g62597caa-py2.py3-none-any.whl

[MisRob]

Thank you @nucleogenesis for taking care of this. It may make more sense to push to #12651 so we can merge together with the KDS version bump that requires this change.

As mentioned, this update is needed too but I worry it won't resolve the tests failure. But if we push related changes to #12651, we can then observe and test more easily.

[MisRob]

Other than that, appendTo.. update looks correct to me. I will do manual QA as soon as it's moved to #12651 (QA is another benefit of having the version bump together with the changes it requires)