Skip to content

Comments

[feat] basic tab completion ui#2240

Closed
JimZhouZZY wants to merge 15 commits intoMoganLab:mainfrom
JimZhouZZY:tab-completion
Closed

[feat] basic tab completion ui#2240
JimZhouZZY wants to merge 15 commits intoMoganLab:mainfrom
JimZhouZZY:tab-completion

Conversation

@JimZhouZZY
Copy link
Contributor

What

  • Implemented an interactive completion listbox that displays available suggestions
  • Added keyboard event binding to support navigation and selection

QQ_1750963674332

How to test your changes

  1. insert a Python code session
  2. input something, and then press tab
  3. you will see the completion listbox as below

@JimZhouZZY JimZhouZZY changed the title [feat] basic qt_completion_listbox [feat] basic tab completion ui Jun 26, 2025
@JimZhouZZY JimZhouZZY force-pushed the tab-completion branch 3 times, most recently from 5dbaa39 to 6b1795d Compare June 30, 2025 12:35
@JimZhouZZY JimZhouZZY changed the base branch from branch-1.2 to main July 3, 2025 05:44
@JimZhouZZY JimZhouZZY force-pushed the tab-completion branch 2 times, most recently from 77903f2 to 2622bdd Compare July 3, 2025 05:46
@JimZhouZZY
Copy link
Contributor Author

Mark:
补全历史这部分应该做错了。逻辑或许应该为:进入补全->退出补全 算一次undo

@JimZhouZZY
Copy link
Contributor Author

JimZhouZZY commented Jul 4, 2025

备忘:

undo/redo还需要完善。

  • backspace的处理需要进一步研究,似乎remove的时候把versioning重设了。
  • 或许archiver需要一个新的变量控制开关,作用效果在versioning之上。

另外有一个bug,复现步骤

  1. 文档1,敲入a ab abc
  2. cmd+n 敲入 a tab
  3. 直接cmd+number
  4. 观察补全框消失,且无法再出现

@JimZhouZZY
Copy link
Contributor Author

Mark:
在只有一个选项自动关闭补全框时输出Warning级别日志

@JimZhouZZY JimZhouZZY force-pushed the tab-completion branch 2 times, most recently from b21eeaf to 6724c30 Compare July 7, 2025 05:23
@JimZhouZZY JimZhouZZY force-pushed the tab-completion branch 14 times, most recently from b8a15cb to 8513ab0 Compare July 7, 2025 08:59
@JimZhouZZY
Copy link
Contributor Author

进行了code clean up。明天转换成小PR

@JimZhouZZY JimZhouZZY closed this Jul 18, 2025
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.

1 participant