[DO NOT MERGE] Remove FocusTrap in favor of KFocusTrap

[lokesh-sagi125]

Summary

I removed all instances of ' FocusTrap ' that were wrapping ' KModal ', as the focus trapping logic is now handled internally within the KModal component. Additionally, I renamed all remaining FocusTrap instances to KFocusTrap, and finally, I deleted the FocusTrap.vue file since it is no longer necessary. These updates ensure focus trapping is handled consistently throughout the system without causing any regressions.

…

References

• #12588
• https://github.com/learningequality/kolibri-design-system/pull/799

[MisRob]

Thank you, @lokesh-sagi125. Is this ready for review? If so, would you please fill in the PR description and also linked the issue this is resolving? It will help reviewers.

[github-actions[bot]]

Build Artifacts

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

[MisRob]

Note related KDS PR https://github.com/learningequality/kolibri-design-system/pull/799. This PR can be merged after a new KDS release with that change is installed in Kolibri.

[MisRob]

I am not entirely sure why would the automatic lint affect another files in this commit, but perhaps it's not an issue. @rtibbles may have an idea perhaps.

[MisRob]

Thanks for getting the PR description ready @lokesh-sagi125. I think this can be reviewed cc @akolson @nucleogenesis @LianaHarris360.

[AlexVelezLl]

Hey @lokesh-sagi125! We just released the changes made in https://github.com/learningequality/kolibri-design-system/pull/799, and updated the KDS version in develop with the new release. You can now rebase this PR on top of develop, and run yarn install to fetch the new KDS version :hugs:

[lokesh-sagi125]

yes @AlexVelezLl will do.