• Home
  • Categories
  • WIKI
  • LANGUAGE: en
    • [Contribute Advice] 这绝对是个bug
    Experiences and Insight 634 views · 8 replies ·
    Tofloor
    poster avatar
    也无风雨也无晴，归去
    deepin
    2023-04-26 22:23
    Author

    这是一个jar包，用压缩工具打开，我截图的地方应该是org，它竟然显示/ ,少了org，当我进入apache时，它显示org，应该是/org/apache。这么明显的错误竟然一直存在，开发质量堪忧啊

    image.png

    Reply Favorite View the author
    All Replies
    xuqi
    deepin testing team
    2023-04-26 22:27
    #1
    • 这个包可以上传一下吗,我们在测试环境复现一下该问题~
    Reply View the author
    Comments
    hotime
    2023-04-28 18:53
    目前深度压缩软件显示的是“返回到：某某上层目录”，而用户可能更希望时刻知晓当前所在目录的路径。毕竟如果只显示上层目录路径，那就不知道当前目录名了。建议参考一下7z或其他压缩软件，单独提供返回按钮以及当前目录路径。
    阿尼樱奈奈
    Moderator
    2023-04-26 22:53
    #2
    It has been deleted!
    DebuggerX
    deepin
    2023-04-26 22:57
    #3

    https://github.com/linuxdeepin/deepin-compressor/blob/500681f7a1b5990fbd2466a33d36e2db69925c38/src/source/tree/treeheaderview.cpp#L39-L42

    void PreviousLabel::setPrePath(const QString &strPath)
{
    QString tmp = '/' + strPath;
    setToolTip(tr("Current path:") + tmp); // 悬停提示


    tmp = tmp.left(tmp.lastIndexOf('/'));
    if (tmp.isEmpty()) {
        tmp.push_front('/');
    }
    QFontMetrics elideFont(tmp);
    QString str = elideFont.elidedText(tmp, Qt::ElideMiddle, width() - 15);
    setText("     .. " + tr("Back to: %1").arg(str));
}

    确实看不懂这个操作joy

    悬停提示的是对的，直接显示的截掉了一级

    Reply View the author
    DebuggerX
    deepin
    2023-04-26 23:30
    #4
    DebuggerX：

    https://github.com/linuxdeepin/deepin-compressor/blob/500681f7a1b5990fbd2466a33d36e2db69925c38/src/source/tree/treeheaderview.cpp#L39-L42

    void PreviousLabel::setPrePath(const QString &strPath)
{
    QString tmp = '/' + strPath;
    setToolTip(tr("Current path:") + tmp); // 悬停提示


    tmp = tmp.left(tmp.lastIndexOf('/'));
    if (tmp.isEmpty()) {
        tmp.push_front('/');
    }
    QFontMetrics elideFont(tmp);
    QString str = elideFont.elidedText(tmp, Qt::ElideMiddle, width() - 15);
    setText("     .. " + tr("Back to: %1").arg(str));
}

    确实看不懂这个操作joy

    悬停提示的是对的，直接显示的截掉了一级

    刚反应过来应该是对的，当前路径是org/ ，所以返回到的路径应该是/……

    Reply View the author
    也无风雨也无晴，归去
    deepin
    2023-04-27 21:46
    #5
    DebuggerX：

    刚反应过来应该是对的，当前路径是org/ ，所以返回到的路径应该是/……

    肯定是bug

    Reply View the author
    hotime
    deepin
    2023-04-28 18:42
    #6

    他这个显示是没有问题的，你目前就是在/org目录中，那么如果返回上级目录的话，上级目录确实是/
    所以显示“返回到：/”并没有错误，不算bug，但肯定是不太符合日常使用习惯的。

    就我个人而言，我在查看压缩包内容时可能更希望时刻知晓目前所在目录的路径，而非上层目录所在路径。就像Windows平台中的7z压缩软件中那样，返回按钮是单独的，在其右侧会给出当前目录全路径，才更直观些。

    毕竟如果像目前这样只知道上层目录而不知道当前目录名，万一点错进了别的目录连个显示的地方都没有，多少肯定是影响使用的。希望开发团队改进。@xuqi

    7z界面.png

    Reply View the author
    也无风雨也无晴，归去
    deepin
    2023-04-29 01:36
    #7
    hotime：

    他这个显示是没有问题的，你目前就是在/org目录中，那么如果返回上级目录的话，上级目录确实是/
    所以显示“返回到：/”并没有错误，不算bug，但肯定是不太符合日常使用习惯的。

    就我个人而言，我在查看压缩包内容时可能更希望时刻知晓目前所在目录的路径，而非上层目录所在路径。就像Windows平台中的7z压缩软件中那样，返回按钮是单独的，在其右侧会给出当前目录全路径，才更直观些。

    毕竟如果像目前这样只知道上层目录而不知道当前目录名，万一点错进了别的目录连个显示的地方都没有，多少肯定是影响使用的。希望开发团队改进。@xuqi

    7z界面.png

    终于有人明白了

    Reply View the author
    xuqi
    deepin testing team
    2023-05-03 21:01
    #8
    • 这个问题确实需要优化一下~
    • 对于用户使用而言显示当前路径更合理一些~
    Reply View the author