一次 github 跟开源大佬的抬杠经历

2023 年 11 月 18 日
 realpg

先叠个盾: 感谢所有开源作者的贡献


不是前端,写个小工具,需要有个简单的界面,于是找了一个比较知名的开源前端 UI 库,非常好上手

几十分钟就写完了,非常不错 完全就按照 sample 拷的代码就行了 因为也只是需要一些基础的功能,显示个 alert ,做个 input 采集用户数据之类

调试时,发现了一个问题,UI 组件之间的排列逻辑有点小问题,于是!important 一把梭 完成任务收工


闲下来了,觉得这问题肯定不是我一个人遇到的,就去了 github 提了个 issue 标签 bug

就像我前面说的,我也不是前端,基本停留在知道 dom 是啥会 getElementById 和理解简单 js 语法的程度

问题所在呢,就是这个 UI 库提供了一个组件,就是整个屏幕都变成半透明灰色没法点,然后中间弹一个类似桌面程序 modal 窗口的的框弹出的是白色的框,产生反差,里面可以自由发挥 html 代码的组件 不知道这玩意你们一般叫啥

这个组件显示是正常的,我完全 copy 的手册的代码,只是把中间的文字替换成了我要显示的文字

然后无意发现,这个组件下面的基本布局组件里有个色块,不会被变灰遮挡,也不会被弹出的 modal 窗口遮挡,modal 窗口正中间有个超级鲜艳的色块

我对 html 还是有点概念的,这应该是下面那个组件里有一部分颜色功能用了 relative/absolute 定位(这玩意不存在用 fixed 定位吧)给了个 z-index 1 导致的问题 相当于类似 floating 的状态

他 z-index 给 1 是因为他想让这个东西显示在他那个组件内部的基准层上面 1 层

而这个白色的框是 display:block 的 普通定位 就到下面去了

这里我最开始犯了个错误 因为这个 modal 状态 F12 不好定位 而 html 是通过前端框架渲染的 代码里写的是模板 所以我漏掉了 直接看那个白色框的 block 的状态 position 是默认值 刚开始以为这个组件整体不是 floating 的状态

提完了 issue 很快维护的大佬就来解答了

他纠正了我的错误分析 告诉我这个组件都是非默认 position 的

然后我又回去仔细找了下,确实,那个背景整体变灰是个 fixed 的 width height 都是 100%的 div 且 z-index 是 1

两个 z-index 都是 1 结果就是按定义顺序了 问题发现

维护的大佬纠正的我的想法时,另外说到: “这是个 z-index 合不合理的问题”

我认为这不合理,我就反问了一句,那你认为这种 modal 功能而论,给 1 的 z-index 到底合不合理?说这话时候,我认为这不合理。我认为这种组件默认应该有一个较高的 z-index 。没必要最高给个 2000 但是高于一般组件 让这个东西确实能遮住大部分组件,应该是正常人类应该有的思维吧?

到这里,我这个职业杠精八代高低杠转世都没想杠什么,您说了,我就一提,委婉的表示了一下我认为这个不太合理,咱是不是要在远期版本改进一下这个默认值 改个 3 或者 5 之类 这就完事了

然后就得到了一个公式化的答复

“它们的层级是一样的,所以最终它们展示上的层级和在文档流的顺序也有关系。修改默认的 z-index 值属于 breaking change 。不过提供了 z-index 的 prop 和 css 变量,你可以根据项目的实际情况进行配置。”

说实话,这玩意我已经改完了,也没用他的方法,因为文档里压根就没写过这玩意要怎么改,变量在哪也没看到,也没有说明。我不觉得一个公开的知名的广泛传播的开源组件,我不过分的基础用法使用它还得去看他源码怎么写的才能正常用

然后我就吐槽了一句,贵司果然大厂风范。然后我就主动 close 了这个 issue 。 毕竟大周末的,不是带薪抬杠,跟大佬在 github 杠不划算……

为啥大厂风范呢?因为这个开源代码在某个大厂的组织下,而不是在个人作者名下。项目简介明确写道是该厂开源产品,且用了该厂商标。

都 closed 了的,该大佬又跑来跟我一顿解释。

可是我就想知道,大佬认为这个到底合不合理,我就杠了一下 我只是想知道你认为到底合不合理

然后又换来了长篇大论的解释……

可是我只是想知道,到底大佬们怎么想的,这种设计到底合不合理。你要觉得合理,你就告诉我合理,以后我也多学习学习怎么设计这种合理。

我就又补充了一句,我只是想知道到底合不合理,能不能用一个字或者两个字告诉我到底合理不合理。我说大厂风范,就是指的这种从不正面面对问题,顾左右耳言他的行为

然后,我就又收获了更长的一篇长篇大论……

这次还用 markdown 给我列了 1 2 3 4

其中第四条是

“默认的值无论是什么都有可能在某个场景下是不合理的。”

我就想知道,现在程序员的群体里,都已经这样了么?话不能直接说,沟通不能简洁有效 什么都得长篇大论,生怕触动了原始写这段代码的巨佬的权威

你觉得合理,你就明确的告诉我合理,我也接受就完事了

60643 次点击
所在节点    程序员
333 条回复
Damenly1
2023 年 11 月 19 日
已经拉黑 OP,
zuston
2023 年 11 月 19 日
OP 确实三观不正
OneOrange417
2023 年 11 月 19 日
挂自己可还行
qiaobeier
2023 年 11 月 19 日
为什么要网暴自己?
lesismal
2023 年 11 月 19 日
@iseki 遇到奇奇怪怪的用户多了,就习惯了人生百态,我刚开始也不理解,遇到一些比 OP 还刁蛮的、心态就慢慢适应了: https://v2ex.ih06.com/t/833524

我估计 OP 是因为自己本身并不是专业的前端、却已经自己搞定了问题、所以心里有点沾沾自喜,所以一时疏忽来了骄傲的毛病。相信很多人也一样、喜欢折腾、小有所成时容易沾沾自喜,我自己也犯过类似的错误,所以觉得正常、劝 OP 平常心

见自己,见天地,见众生,谦逊难得
jeesk
2023 年 11 月 19 日
@lesismal 这个正如我贴了 bootstrap 的文档一样, 可能开源项目因为一些约定或者历史问题导致这些配置并不合理, 项目维护者也清楚, 但是楼主强行要求修改,这根本没有考虑到项目的兼容性的问题。 反而给别人扣上一顶大厂开源的帽子, 我个人觉得这是不礼貌的行为。开源项目始终要站在社区维护者的角度。 不是自己想怎么搞就怎么搞, 如果可以那你自己开一个分支自己玩呗。
byzod
2023 年 11 月 19 日
@LonnyWong
点进去之前脑补的场景:技术问题交流不畅,来回拉扯百十回合,3 页之后恼羞成怒开喷

实际的场景:聊了 3 局,开喷

喷就罢了,还挂自己,多少有点大病……人类的多样性真不是吹的🤣
byzod
2023 年 11 月 19 日
局→句
话说 v2 评论不能 markdown 就罢了,怎么连编辑都没有啊(恼
alfchin
2023 年 11 月 19 日
杠精 op 还觉得自己有理?
走好不送,祝你次次用开源次次翻车
zylll520
2023 年 11 月 19 日
好家伙,还有你这样的人呀
chenyu0532
2023 年 11 月 19 日
自己的沟通问题。在这种只能用文字沟通的时候,肯定不能用能让别人产生负面情绪的词,即使大佬也是有脾气的。这东西跟线下交流很不一样,线下可能会带有表情/语气等等,别人会知道你真实的意思,线上人家知道你是啥人,到底对面说这话纯粹是玩笑还是嘲讽
Pastsong
2023 年 11 月 19 日
以后换个匿名账号吧,这两个已经废了
F7ionsy
2023 年 11 月 19 日
已拉黑。
buf1024
2023 年 11 月 19 日
itechify
2023 年 11 月 19 日
人家的意思是这样是合理的,需要不同场景去定制化
hikarugo
2023 年 11 月 19 日
看了原贴,只能说那个维护者是真的好,别人是想让你知其然的基础上,给你解决方案,让你自己做选择,你倒好,自己在那里气急败坏,说白了就是让人认同你,v2 发帖同理,自己发帖的时候甚至还没意识到自己的问题,也是一种天赋
beiyu
2023 年 11 月 19 日
vant 没问题,OP 纯二极管。🤡
xiaochena
2023 年 11 月 19 日
看到大家都在骂你,我就放心了。
a67793581
2023 年 11 月 19 日
我也是开源作者,老实说默认值在各种场景下都有可能是不合理的,而你刚刚好碰到了有没找到配置文档。就这么简单,你唯一需要做的就是帮作者完善文档而不是发牢骚
zsdroid
2023 年 11 月 19 日
笑死,这么多人说 op ,op 也不反思反思,还在嘴硬。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://v2ex.ih06.com/t/993100

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX