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

增强列表模式下重命名时的色彩对比 #60

Closed
dongfengweixiao opened this issue Oct 30, 2018 · 9 comments
Closed

增强列表模式下重命名时的色彩对比 #60

dongfengweixiao opened this issue Oct 30, 2018 · 9 comments
Assignees

Comments

@dongfengweixiao
Copy link

_virtualbox machine_20181030223519
_virtualbox machine_20181030223509
_dde-file-manager_20181030223819
_dde-file-manager_20181030223746

对比 Windows 下的列表模式文件被选中和文件处于改名状态的样式,可以发现,不管在文件名的长度如何,都可以明确的显示其处于选中状态还是改名状态。
但是对比 DDE 的文件管理器,在选中时使用的是浅蓝色,当变为rename状态时,底色变为深蓝色,但是文字的背景依旧是浅蓝色,且和背景的深蓝色之间仅有很小的间隔。当文件名的长度不是很长时,并没有太多的影响,但是当文件名的长度很长(超过可显示的长度)就会导致用户无法直观的理解当前状态。
_dde-file-manager_20181030224539

@dongfengweixiao dongfengweixiao changed the title 增强列表模式下 rename 时的色彩对比 增强列表模式下重命名时的色彩对比 Oct 30, 2018
@wangjia12
Copy link

@zccrs 刚找 @shule1987 看了,说这个是bug。找他确认效果吧。

@wangjia12 wangjia12 assigned zccrs and unassigned wangjia12 Nov 1, 2018
@wangjia12 wangjia12 added bug | functional behavior bug | 功能行为的 and removed bug | functional behavior bug | 功能行为的 labels Nov 1, 2018
@shule1987
Copy link

@zccrs 把列表模式重命名的文字选中的背景颜色改成rgba(255,255,255,0.4)

@dongfengweixiao
Copy link
Author

理论上,在宫格模式下,如果文件名的长度恰好充满N行,则也比较难以区别(当然了,在宫格模式下,整个文件名都在视野范围内,所以不更改也没用关系)。

@zccrs zccrs assigned BLumia and unassigned zccrs Nov 13, 2018
@BLumia
Copy link
Member

BLumia commented Nov 20, 2018

@BLumia BLumia assigned shule1987 and unassigned BLumia Nov 20, 2018
@zccrs
Copy link
Member

zccrs commented Nov 22, 2018

@shule1987 看一下现在的效果对不对

@shule1987
Copy link

shule1987 commented Nov 22, 2018 via email

@BLumia
Copy link
Member

BLumia commented Nov 22, 2018

@shule1987 : 昨天装了上面的 cr。没看到效果

因为你给的颜色就是这样的.......

@shule1987
Copy link

已经验证

@BLumia BLumia assigned liuwen123 and unassigned shule1987 Nov 23, 2018
deepin-gerrit pushed a commit that referenced this issue Nov 23, 2018
#60

Change-Id: Ie95fc357c0253f254788276ac6b0e62ce68944af
@liuwen123
Copy link

已修复

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

No branches or pull requests

6 participants