search

PointTeam / PointDownload

homepage
GNU General Public License v3.0
224 stars 43 forks source link

修改notice窗口为动态高度，以完全显示所有文字 #14

Closed sbwtw closed 9 years ago

sbwtw commented 9 years ago

修改notice窗口为动态高度，以完全显示所有文字和标题，还有一些其它小改动

PointTeam commented 9 years ago

将QUrl转为QString的已经改了，把那两个commit丢弃吧

sbwtw commented 9 years ago

我已经pull过改qurl转string的代码了，可以直接合并的

On Wed, Jan 7, 2015, 09:55 point-team notifications@github.com wrote:

将QUrl转为QString的已经改了，把那两个commit丢弃吧

— Reply to this email directly or view it on GitHub https://github.com/PointTeam/PointDownload/pull/14#issuecomment-68967978 .

PointTeam commented 9 years ago

好的

Match-Yang commented 9 years ago

实在不好意思，前端时间没有空，没做测试，今天才测试，发现notice窗口高度改的有问题 截图

sbwtw commented 9 years ago

能不能把这个下载地址给我测试一下？ 在pointdownload/main.cpp 第93行有个测试的代码，取消这里的注释显示出来会有问题吗？

Match-Yang commented 9 years ago

你好，我测试了你最新的commit，有两个问题： 1、在已完成列表和垃圾桶列表的重新下载按钮都不能用； 2、正在下载列表，某项完成下载后，并没有从正在下载列表中删除对应的项，应该是漏了qml那边的数据更新了 请看一下你重构qml代码的时候漏掉什么没有

sbwtw commented 9 years ago

我又做了修改，过几天再commit

On Sun, Mar 8, 2015, 22:26 杨万青 notifications@github.com wrote:

你好，我测试了你最新的commit，有两个问题： 1、在已完成列表和垃圾桶列表的重新下载按钮都不能用； 2、正在下载列表，某项完成下载后，并没有从正在下载列表中删除对应的项，应该是漏了qml那边的数据更新了 请看一下你重构qml代码的时候漏掉什么没有

— Reply to this email directly or view it on GitHub https://github.com/PointTeam/PointDownload/pull/14#issuecomment-77751667 .

Match-Yang commented 9 years ago

好的