帮Netpole Review了一下RichEditor控件

RichEditor代码Review。
1、Resources目录下的图片可以分下类,把表情和工具栏图片分开。
2、咱们的代码要用统一的命名空间,用WawaSoft开头,像你这个控件就要用WawaSoft.WinControls开头,其它的命名空间我一会儿写个列表。
3、给文字设置了一定的格式之后,回车换行后应该取消格式吧,要不还得手工把格式都去掉才可以,不知道这是by desgin还是bug,你看下word是怎么处理的。
4、前景色和背景色如果上次选择的是蓝色,下次如果不点击下拉颜色,应该默认就是蓝色,而且一点就可以应用,而不是再次弹出颜色选择器,点击三角再显示颜色选择器。
5、我选定文字后,点设置字体,怎么好多英文字体选择了后字体不改变呀,你确认下是不是bug。另外我觉得字体不用都列出来,好多字体咱不用,你说呢。
6、建议加上清楚格式的功能,就是把字号大小,颜色等都还原为默认值,当然是选定的文字哦
7、研究一下能否做出格式刷的功能,呵呵。
8、数字列表的功能还没有实现
9、你这全部替换功能做的有问题,我一点替换,动态的给我添加了N多字符,逻辑可能有问题,再完善一下把。
10、工具栏上的图片和查找按钮没有设置tootip,你再看看还有哪些没有设置的,另外像格式栏我觉得以后可以放到主框架里,像vs.net那样,可以控制显示哪些工具条,工具条和下面控件再交互,估计不好实现,以后再说。
11、加入插入时间戳的功能
12、看看可不可以加表格相关功能。
13、当出现查找面板的时候,再次点击工具栏的查找按钮,应该隐藏掉。
14、你查找里的全部是什么意思呀?
15、当编辑器托拽变小的时候,查找栏成半截了,应该控制控件的最小宽度。
16、可否考虑加入光标切换功能,比如文档特别长的话,能记住上次光标停留的位置,拖动滚动条到最下面的时候,可以通过按钮或者快捷键快速定位到上次光标停留的位置,ultraedit和delphi都有这个功能,当然这个可能要用个堆栈来实现,和undo的原理差不多,你只要保存最近10次的光标位置就行了。

先帮你分析到这儿吧,代码写的不错,控件的命名也比较符合规范,我没有详细看代码,只是黑盒测试了一把,有空再review你的代码。

posted @ 2007-04-28 00:35  蛙蛙王子  Views(503)  Comments(0Edit  收藏  举报