fisheva / Eva-Theme

A comfortable and semantic theme.
https://marketplace.visualstudio.com/items?itemName=fisheva.eva-theme
MIT License
452 stars 38 forks source link

几个小意见 #30

Closed roy-tian closed 4 years ago

roy-tian commented 4 years ago

写C/C++多些,对深色theme提几个小小的意见:

  1. 80字符的ruler是亮蓝色,与背景略显违和;
  2. 无法为CMakeLists.txt着色;
  3. C++中未对 template和typename 着色(不知是不是有意为之)

总之楼主这套theme奶香奶香的,用起来很舒服很喜欢!

fisheva commented 4 years ago

谢谢支持~ :D

1,没懂这个意思, 可以上段代码吗?最好是截图+代码片段

后面2条需求get到了, CMake我还要去了解下。

roy-tian commented 4 years ago

就是这根线 就是这根线,个人觉得可以略微调淡一点。仅供参考:)

fisheva commented 4 years ago

请更新Eva Theme至0.7.9最新版本。

阅读下面回答的预备了解: 按下"Ctrl+Shift+P"(Window) 或 "Cmd+Shift+P"(Mac) 选中 "Inspect TM Scopes" 可以查看所有字符的Scope值。

回答您的问题: 1, 已调淡。 2, 阅读一篇新手教程后, 我大致了解了CMakeLists.txt是什么东西。

首先VSCode对txt文档文件是没有字符识别算法支持的, 即没给txt文件里的字符添加Scope值, 主题插件也就无法对txt文档文件里的字符着色。 image

但如果将txt文件命名为CMakeLists.txt, 它在VSCode里的"文件类型"(引号是因为后缀没变)居然就变了(感叹下txt文件的神奇), 并且有了一定字符识别算法支持。 image

遗憾的是支持程度还很粗浅, 如图, 符号括号字符连成一串, 无法细分。 image

另外CMakeLists.txt这个命名也很严苛, 无视大小写, 其它任何字符差别都会被别VSCode识别成txt文件, 失去字符识别算法支持。 image

综上这个问题的回答是, 已添加CMakeLists.txt文件着色, 受限于算法支持, 还很简陋。

3,已着色, 不是有意为之, 遗漏。


这次调试C++文件, 发现一个特别尴尬的情况, 微软官方推荐的C/C++插件会导致部分字符颜色渲染错误。 image 这种渲染错误不是仅与Eva Theme冲突, 是会影响到所有主题插件。我已经向其提交了一个issue

fisheva commented 4 years ago

cpptools团队回复很快, 有解决办法了, 关闭cpptools对部分字符渲染影响的方式是在setting里搜索colorization, 选择Disabled这个选项。 image

roy-tian commented 4 years ago

作者很用心,很是让人感动!如果我是个土豪,肯定会给你donate 8000块。 不过 cpp 插件还是不要关闭增强着色选项,他那个我感觉不算着色错误,留着的话会根据语义提供更丰富的着色。(不过貌似官方这个选项有bug,会偶尔失灵,重新打开vscode又可能会好,不知道这里有什么我不知道的魔法) 总之作者这么用心肯定会成为享誉国内外的 带 工 程 师。请务必 加 带 力 度。