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

switch key list font-family to consolas and support key name indent #434

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

magictext
Copy link

切换键名字体为consolas,便于观察
支持键名缩进,查询更加方便

@magictext
Copy link
Author

效果演示

@qishibo
Copy link
Owner

qishibo commented Feb 2, 2021

感谢pr,不过
1、字体在全局设置里是可以更改的,没必要写死
2、分隔符也是可以在编辑链接里自定义的,默认冒号,展示时也没必要写死

@qishibo
Copy link
Owner

qishibo commented Feb 2, 2021

还有就是不太明白你说的键名缩进是什么意思

yu_jiang added 2 commits February 2, 2021 14:20
…opManager

# Conflicts:
#	src/util.js
适配最新代码,并进行简单测试
@magictext
Copy link
Author

已适配最新代码.
已删除对字体的硬修改.
已添加对分隔符的适配
键名缩进的效果可以参考此图片https://s3.ax1x.com/2021/02/02/ymfVIS.png

@qishibo
Copy link
Owner

qishibo commented Feb 2, 2021

目前效果是这样的
image

看你的代码是子级只有一个文件夹时直接和上级名称合并?

@magictext
Copy link
Author

简单来说是的.
就是一个ide的目录缩进功能

@qishibo
Copy link
Owner

qishibo commented Feb 5, 2021

想了想,有逻辑问题
key列表非全部加载,比如只加载出了篮圈的,红圈的未加载,
此时如果合并为zzz:bbb,则文件夹右键可以删除zzz:bbb下的所有key,但此时zzz已经合并了,如果想删除zzz下的所有key则无法实现 。先放着,看后续有没有缩进这方面的需求再适配

image

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

Successfully merging this pull request may close these issues.

None yet

2 participants