Shimada666 / v-code-diff

A vue code diff display plugin, support Vue2 / Vue3
https://shimada666.github.io/v-code-diff/
MIT License
407 stars 68 forks source link

增加支持 ignoreMatchingLines 属性 #121

Closed Lruihao closed 10 months ago

Lruihao commented 10 months ago

Description

灵感来自命令 diff file1.json file2.json --ignore-matching-lines="time", 增加行关键词忽略匹配功能,side-by-side 模式生效

Resolved #120

Preview

设置 ignoreMatchingLines="time",效果如下:

image
Shimada666 commented 10 months ago

加这个倒是ok,但是页面上我感觉应该不用显示 ignoreNum?我第一眼其实没看明白那个正负 lines 是啥意思..如果有需要定制插槽感觉就 ok。另外那个 ignoreNum typescript 类型标注有问题,可以改一下吗,和 addtionNums 一样平铺展示就挺好。顺便需要支持下 line-by-line 模式

Lruihao commented 10 months ago

加这个倒是ok,但是页面上我感觉应该不用显示 ignoreNum?我第一眼其实没看明白那个正负 lines 是啥意思..如果有需要定制插槽感觉就 ok。另外那个 ignoreNum typescript 类型标注有问题,可以改一下吗,和 addtionNums 一样平铺展示就挺好。顺便需要支持下 line-by-line 模式

  1. 我当时加 ignore lines的时候也感觉略有不妥,那就默认不显示吧,在 stat 插槽参数保留 ignore 的相关参数,有需要的用户自行处理。
  2. ignoreNum 平铺的话要拆成 ignoreAdditonsNum 和 ignoreDeletionsNum 吗?
  3. line-by-line 我明天改改
Shimada666 commented 10 months ago

是的,直接拆成这个吧

Lruihao commented 10 months ago

是的,直接拆成这个吧

ok, 已修订。

顺便把 filename 和 newFilename 的字体大小统一为了 12px(原来是 filename 13px newFilename 12px,大小不一致看着有点点别扭 ...)

Shimada666 commented 10 months ago

蛮好的,感谢。我晚点再测一下,无问题我发个版本