1111mp / nvm-desktop

Node Version Manager Desktop - A desktop application to manage multiple active node.js versions.
https://github.com/1111mp/nvm-desktop
MIT License
621 stars 31 forks source link

页面样式有点问题 #60

Closed lsrweb closed 4 months ago

lsrweb commented 5 months ago

image 路径文本展示不全

另外,这下图右边俩按钮是就是这样设计小一号的嘛 image

不好生意,做前端测试多了,感觉有点怪 但是总体来说,99分

1111mp commented 5 months ago

另外,这下图右边俩按钮是就是这样设计小一号的嘛

是的 按钮是高度是故意小一点的(24px),左边的输入框和View高度为32px。目的是想将按钮和table操作分开。但是你这么一说,确实有点不协调(可能是设置成简体中文情况下),我后续统一设置成24px

Screenshot 2024-02-13 at 21 06 41

或许32px会更好?

Screenshot 2024-02-13 at 21 07 23

路径文本展示不全

是本身路径有空格还是显示的时候会这样 麻烦能够将完整的路径复制出来看一下吗 我本地测试一下 如果有问题的话应该是这一行的代码有问题:/src/renderer/src/pages/projects/index.tsx#L196

不好生意,做前端测试多了,感觉有点怪

没关系,反而非常欢迎。因为目前所有的ui都是自己凭主观感觉实现的,也没人参考,所以难免会有一些问题,还请谅解。

lsrweb commented 5 months ago

D:\DevCode___\A_XJ_YingFeng\sub-zljc 这是项目路径,应该不是这句代码的问题

32px那个大点的我觉得挺好

1111mp commented 5 months ago

好的 感谢 🌹

已经定位到问题了,元素行高太低造成的(css样式。 不仅仅是下面的下横线,有些字母(比如g)下面都被遮住了,我会在下个版本修复这个问题。

32px那个大点的我觉得挺好

👌 会随着下个小版本一起发布 非常感谢。

不过目前自动化测试还有点问题 得等到webdriverio这边一个issue修复再发出下个版本 所以可能不会那么快 还望见谅

lsrweb commented 5 months ago

期待更好,一起进步 image

1111mp commented 4 months ago

这个问题目前已经在最新的 Release v3.1.0 中修复

那么我就先把这个issue给关掉了 当然后续如果有任何问题都可以随时重新打开 谢谢

lsrweb commented 4 months ago

好哒!!Thanks♪(・ω・)ノ谢谢,元宵节快乐

1111mp commented 4 months ago

谢谢,也祝你元宵节快乐🏮