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

标尺等功能和部分缺陷修复 #332

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

sln-1550
Copy link

1、增加了标尺
2、HEX模式下左右窗口复制和粘贴的自动适配,左边窗口复制粘贴HEX数据,右边复制粘贴原始数据,同时也去掉了hex右键复制相关的3个菜单
3、当前行高亮用背景方式而不是线框
4、把FindTextAll参数showResult改成keepResult,表示是否不删除之前的查找结果
5、增加了一些python内置模块等
6、修复了搜索框调节不为0,每次都变大的BUG

asmwarrior and others added 3 commits February 22, 2024 09:22
2、HEX模式下左右窗口复制和粘贴的自动适配,左边窗口复制粘贴HEX数据,右边复制粘贴原始数据,同时也去掉了hex右键复制相关的3个菜单
3、当前行高亮用背景方式而不是线框
4、把FindTextAll参数showResult改成keepResult,表示不删除之前的查找结果
5、增加了一些python内置模块等
@LiMinggang LiMinggang self-assigned this May 17, 2024
@LiMinggang LiMinggang added this to the 0.4.21 milestone May 17, 2024
@LiMinggang
Copy link
Owner

我会把这些修复、新功能拆分成独立的commits

@LiMinggang
Copy link
Owner

LiMinggang commented May 18, 2024

我把Ruler的cursor的xpos给改成了Column,因为指示图像意义的座标没有太大意义,而Column是字符座标。对于非固定宽度字符显示会有问题,这个加上了告警

@LiMinggang
Copy link
Owner

LiMinggang commented May 18, 2024

还有一些小改进和优化,都合入了
3059dd5
37c7ab4

@LiMinggang
Copy link
Owner

LiMinggang commented May 18, 2024

修复了搜索框调节不为0,每次都变大的BUG
87941c6

@LiMinggang LiMinggang force-pushed the master branch 2 times, most recently from 44982b1 to 87941c6 Compare May 18, 2024 05:55
@LiMinggang
Copy link
Owner

Python
9df94c1

@LiMinggang LiMinggang force-pushed the master branch 6 times, most recently from 2d429e7 to 4b84ec6 Compare May 20, 2024 07:00
@sln-1550
Copy link
Author

辛苦了,确实标尺我只考虑了一种字体的尺寸

@LiMinggang LiMinggang force-pushed the master branch 2 times, most recently from 4a83244 to 7a553b4 Compare May 21, 2024 02:50
@LiMinggang
Copy link
Owner

没啥,有很多细枝末节要熟悉代码才知道。我把颜色和字体都设成wx控件的颜色和字体,然后第一次画的时候调整了一下尺寸。
你的设计思路很好了。
其他的部分,比如高亮的和copypaste那部分,我再想想看。

@sln-1550
Copy link
Author

最近发现一个bug,当前行高亮时碰到关键字高亮就会停掉,没有高亮行尾剩余部分。这块我自己改好了,等你这边融合以后再提交吧

@LiMinggang
Copy link
Owner

我这边大部分问题都修好了,linux下面也测试通过了。目前就是python那个修改需要回退,因为linux下编译有些依赖函数找不到,链接失败。我会保留提交记录,待以后解决。

@sln-1550
Copy link
Author

sln-1550 commented May 28, 2024

那我这边还需要做什么么?第一次PR,没啥经验

@LiMinggang
Copy link
Owner

你不用等,我现在对标尺的修复已经比较有信心了:windows和linux都测试过了。
所以如果你愿意,可以同步一次代码,然后把剩余的修改提上来。然后基于我的修改继续使用中测试,看看还有没有bug。
对于其他部分我目前的想法是:

  1. 当前行的显示如果可以,应该也是给用户提供选择。
  2. 下面这个只是改了个名字吗?
    4、把FindTextAll参数showResult改成keepResult,表示是否不删除之前的查找结果

@LiMinggang
Copy link
Owner

然后自适应copy/paste那个,我是没太明白。
最初我的设计思路是:copy的内容可以变成 0E 0A 1D这种串,而不是只能贴内容。因为有时候需要在查找和插入的时候用到十六进制。
目前你的设计是在文本状态就贴内容?十六进制输入时贴十六进制串?

@sln-1550
Copy link
Author

sln-1550 commented Jun 1, 2024

你不用等,我现在对标尺的修复已经比较有信心了:windows和linux都测试过了。 所以如果你愿意,可以同步一次代码,然后把剩余的修改提上来。然后基于我的修改继续使用中测试,看看还有没有bug。 对于其他部分我目前的想法是:

  1. 当前行的显示如果可以,应该也是给用户提供选择。
  2. 下面这个只是改了个名字吗?
    4、把FindTextAll参数showResult改成keepResult,表示是否不删除之前的查找结果

不是,因为我之前写脚本的时候,发现脚本执行FindTextAll后结果不会自动展开,所以才加了这个参数,之前的showResult我感觉用不上,不会用到不展示结果的搜索。

然后自适应copy/paste那个,我是没太明白。 最初我的设计思路是:copy的内容可以变成 0E 0A 1D这种串,而不是只能贴内容。因为有时候需要在查找和插入的时候用到十六进制。 目前你的设计是在文本状态就贴内容?十六进制输入时贴十六进制串?

是这个意思,就是选中左边的HEX区域,复制粘贴都是经过HEX编解码的;而在右边的复制粘贴就是原来的复制粘贴

@LiMinggang
Copy link
Owner

LiMinggang commented Jun 1, 2024

你不用等,我现在对标尺的修复已经比较有信心了:windows和linux都测试过了。 所以如果你愿意,可以同步一次代码,然后把剩余的修改提上来。然后基于我的修改继续使用中测试,看看还有没有bug。 对于其他部分我目前的想法是:

  1. 当前行的显示如果可以,应该也是给用户提供选择。
  2. 下面这个只是改了个名字吗?
    4、把FindTextAll参数showResult改成keepResult,表示是否不删除之前的查找结果

不是,因为我之前写脚本的时候,发现脚本执行FindTextAll后结果不会自动展开,所以才加了这个参数,之前的showResult我感觉用不上,不会用到不展示结果的搜索。

然后自适应copy/paste那个,我是没太明白。 最初我的设计思路是:copy的内容可以变成 0E 0A 1D这种串,而不是只能贴内容。因为有时候需要在查找和插入的时候用到十六进制。 目前你的设计是在文本状态就贴内容?十六进制输入时贴十六进制串?

是这个意思,就是选中左边的HEX区域,复制粘贴都是经过HEX编解码的;而在右边的复制粘贴就是原来的复制粘贴

那这个功能跟原来的设计还不是同一个应用场景。可以考虑两者共存

@sln-1550
Copy link
Author

sln-1550 commented Jun 3, 2024

你不用等,我现在对标尺的修复已经比较有信心了:windows和linux都测试过了。 所以如果你愿意,可以同步一次代码,然后把剩余的修改提上来。然后基于我的修改继续使用中测试,看看还有没有bug。 对于其他部分我目前的想法是:

  1. 当前行的显示如果可以,应该也是给用户提供选择。
  2. 下面这个只是改了个名字吗?
    4、把FindTextAll参数showResult改成keepResult,表示是否不删除之前的查找结果

不是,因为我之前写脚本的时候,发现脚本执行FindTextAll后结果不会自动展开,所以才加了这个参数,之前的showResult我感觉用不上,不会用到不展示结果的搜索。

然后自适应copy/paste那个,我是没太明白。 最初我的设计思路是:copy的内容可以变成 0E 0A 1D这种串,而不是只能贴内容。因为有时候需要在查找和插入的时候用到十六进制。 目前你的设计是在文本状态就贴内容?十六进制输入时贴十六进制串?

是这个意思,就是选中左边的HEX区域,复制粘贴都是经过HEX编解码的;而在右边的复制粘贴就是原来的复制粘贴

那这个功能跟原来的设计还不是同一个应用场景。可以考虑两者共存

我觉得是一个场景,就是复制和粘贴HEX编、解码后的内容

@LiMinggang
Copy link
Owner

你不用等,我现在对标尺的修复已经比较有信心了:windows和linux都测试过了。 所以如果你愿意,可以同步一次代码,然后把剩余的修改提上来。然后基于我的修改继续使用中测试,看看还有没有bug。 对于其他部分我目前的想法是:

  1. 当前行的显示如果可以,应该也是给用户提供选择。
  2. 下面这个只是改了个名字吗?
    4、把FindTextAll参数showResult改成keepResult,表示是否不删除之前的查找结果

不是,因为我之前写脚本的时候,发现脚本执行FindTextAll后结果不会自动展开,所以才加了这个参数,之前的showResult我感觉用不上,不会用到不展示结果的搜索。

然后自适应copy/paste那个,我是没太明白。 最初我的设计思路是:copy的内容可以变成 0E 0A 1D这种串,而不是只能贴内容。因为有时候需要在查找和插入的时候用到十六进制。 目前你的设计是在文本状态就贴内容?十六进制输入时贴十六进制串?

是这个意思,就是选中左边的HEX区域,复制粘贴都是经过HEX编解码的;而在右边的复制粘贴就是原来的复制粘贴

那这个功能跟原来的设计还不是同一个应用场景。可以考虑两者共存

我觉得是一个场景,就是复制和粘贴HEX编、解码后的内容

就一个问题:在文本模式下,如果用户粘贴,你觉得他想粘贴什么内容?

@sln-1550
Copy link
Author

sln-1550 commented Jun 5, 2024

就一个问题:在文本模式下,如果用户粘贴,你觉得他想粘贴什么内容?

文本模式没有改,我的代码只是在HEX模式下工作。
这个我不想跟你争辩,毕竟我是hex模式用的多,我用自己改的版本觉得方便,你这边可以不加,没关系的

@sln-1550
Copy link
Author

20240822_144117.mp4

贴个视频说明下吧,省的讲不清楚。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants