Skip to content

Conversation

@gowrimanoharir
Copy link

@gowrimanoharir gowrimanoharir commented Nov 11, 2025

Closes-#7713

Background

Looks like the default code mirror behavior was overridden in the original PR when autocomplete was implemented, to not allow tab key to autocomplete

#112

Changes

Removing the override so it reverts to default code mirror behavior allowing tab to autocomplete suggestions

Tested in local, can upload videos if needed.

@CLAassistant
Copy link

CLAassistant commented Nov 11, 2025

CLA assistant check
All committers have signed the CLA.

@gowrimanoharir gowrimanoharir changed the title fix: removed tab override in codemirror extension removed tab override in codemirror extension Nov 11, 2025
@jackkav jackkav requested a review from pavkout November 25, 2025 16:49
Copy link
Contributor

@pavkout pavkout left a comment

Choose a reason for hiding this comment

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

I tested it, and everything works as expected. Thanks for your help.

@pavkout pavkout force-pushed the 001-tab-autocomplete branch from 7ccd81f to 2dfe879 Compare November 27, 2025 17:11
@gowrimanoharir
Copy link
Author

@pavkout thanks for the approval, the workflow still says Test Expected please let me know I any should be done 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