首页   注册   登录
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
Persephone
V2EX  ›  程序员

Ant Design 圣诞地雷

  Persephone · 2018-12-25 09:55:58 +08:00 · 31974 次点击
这是一个创建于 587 天前的主题,其中的信息可能已经有所发展或是发生改变。

https://github.com/ant-design/ant-design/issues/13098#issuecomment-449754024

做框架不好好修 bug,掺 x 为什么这么积极?

问了一圈,我认识的前辈里 9 月 19 号就已经发现了这段“菜( di )单( lei )”

https://github.com/ant-design/ant-design/commit/00aebeb9756afecc884ad48486084836b9a2707a

始作俑者甚至还表示:changelog 里无须体现( https://github.com/ant-design/ant-design/commit/00aebeb9756afecc884ad48486084836b9a2707a#commitcomment-30445378

现在爆发出来我觉得非常正常,因为开发者既然用你这个库就是信你这块牌子,很少人一个个文件审查、一个个提交的看。

各位怎么看?我是觉得非常不合适。

第 1 条附言  ·  2018-12-25 10:36:50 +08:00

https://github.com/ant-design/ant-design/issues/13818#issuecomment-449753026 早就做好了被骂的准备

怕不是只要做好坐牢的准备,什么恶意代码都敢掺咯?

第 2 条附言  ·  2018-12-25 13:53:26 +08:00

这里想多说两句。

首先,Ant Design 对于前端业界的贡献是很大的,虽然这次出现了这样的人为疏失,但不得不说这个项目已经是为数不多完整可用、并且不断在迭代的项目了。我相信大多数人就算在使用中遇到这样那样的问题,最终还是会感谢这个项目的开发人员做出的贡献的。

那问题出在哪儿?你不能要求所有开发者都逐行审查自己使用的库的代码,因为这在一半工程实践里不现实。你也不能要求开源项目的作者付出120% 的心力来完善项目保证没有问题,因为这也不现实。

怪谁呢?想来想去,最后发现还是国人的开源模式和做法上的问题。比如我在主楼提到的,已经有大佬提早在审查代码的时候发现了,但没有提交pr 和issue,只是在自己的项目里进行了相关的修正,最终没有把问题扼杀在摇篮里。问起大佬为什么不提交?大佬的回答:“反正这不是我的库,干什么多管闲事?”。联想起到现在还有很多朋友发提问贴,得到回复并解决之后就编辑掉帖子的做法,突然就感觉很无力。

是啊,这就是现实。

  1. 希望开发者不要再进行独断专横的操作了,您在开发的代码被很多前端项目依赖,请不要辜负使用者对您的信任。

  2. 希望各位开发者发现问题或者解决问题的时候,请不要吝啬分享,说不定你做出的贡献可以让很多人收益。

  3. 在使用各种服务的各种功能前,请务必摸清其中规则。比如:github 的issue 并不是用来骂街和观光的,npm 不是用来传乱七八糟东西的。

  4. 你索取是你的权力,但别人提供并不是义务。

生活不易,愿国内开源氛围能够越来越好吧。

262 条回复    2018-12-26 14:29:15 +08:00
1  2  3  
SCaffrey
    1
SCaffrey   2018-12-25 10:05:32 +08:00 via Android
Justin13
    2
Justin13   2018-12-25 10:07:24 +08:00 via Android
正常,官方教程里面的 datepicker 导入方法,有一个完全导不进来。绝了,我他妈那天加班好久翻代码才发现得从 es 文件夹里导。
jianzhihao1996
    3
jianzhihao1996   2018-12-25 10:08:03 +08:00
看到被炒的老哥莫名心痛
hdr01
    4
hdr01   2018-12-25 10:11:07 +08:00   ❤️ 1
console 里玩彩蛋就行了,这种行为就是在代码里下毒!
qhxin
    5
qhxin   2018-12-25 10:12:33 +08:00
哈哈,今天看到了
Vegetable
    6
Vegetable   2018-12-25 10:14:08 +08:00
输出爆炸
ericgui
    7
ericgui   2018-12-25 10:14:22 +08:00
@jianzhihao1996 什么情况?
Vegetable
    8
Vegetable   2018-12-25 10:15:48 +08:00
@ericgui 一个老哥表示因为这个彩蛋被公司开除了.
fe619742721
    9
fe619742721   2018-12-25 10:16:16 +08:00
天秀操作
hlwjia
    11
hlwjia   2018-12-25 10:18:57 +08:00   ❤️ 1
issue 里面有条 comment 很值得思考

"I really want to use Ant for future projects but decisions like this concern me."

先不说修不修 bug 的问题,但是这个彩蛋作为一个默认出现、没有设置可以配置、需要用 !important 的方式来 overwrite 的决定,是怎么通过需求审核、然后代码审核、然后再合并的?

Ant Design 的团队应该意识到你们的产品已经不是小打小闹了,要有国际水准。
airyland
    12
airyland   2018-12-25 10:20:55 +08:00 via iPhone   ❤️ 1
不支持,不理解,所谓的彩蛋对别人就是事故。
maichael
    13
maichael   2018-12-25 10:22:05 +08:00
愚蠢到没话说。
zhuoyan
    14
zhuoyan   2018-12-25 10:22:37 +08:00
秀啊
ericgui
    15
ericgui   2018-12-25 10:24:06 +08:00
@Vegetable 卧槽,阿里的人都是些什么东西啊。这不是坑人么
oott123
    16
oott123   2018-12-25 10:25:03 +08:00
“早就做好了挨骂的准备”

珍爱生命,拒绝 ant design,毕竟你可没有做好挨骂的准备。
hitfm
    17
hitfm   2018-12-25 10:28:59 +08:00
修复方案也是牛逼啊,直接样式 override 掉~
guoshencheng1
    18
guoshencheng1   2018-12-25 10:30:30 +08:00
在用开源框架之前没有自己看过源码,在重要的项目里使用这样的框架,也要考虑一下自己的问题吧
wsstest
    19
wsstest   2018-12-25 10:31:49 +08:00
心疼那些被这个菜单祸害的开发们
shylockhg
    20
shylockhg   2018-12-25 10:34:02 +08:00
@jianzhihao1996 真的是因为这个?
WuwuGin
    21
WuwuGin   2018-12-25 10:34:11 +08:00
@guoshencheng1 这种根本不属于框架的范畴了,框架开箱即用也是特点,况且没有文档说明,你是看了多少框架的所有源码?
LevineChen
    22
LevineChen   2018-12-25 10:35:44 +08:00   ❤️ 42
@guoshencheng1 专业! 就应该像这位老哥一样, 通读项目源码. 不然出了 bug 就是你们自己的问题. 另外 chromium 源码看到哪了?
FunnyCodingXu
    23
FunnyCodingXu   2018-12-25 10:35:53 +08:00   ❤️ 3
@guoshencheng1 老哥前端后端? 所有库的源码你都看过?
maemual
    24
maemual   2018-12-25 10:37:02 +08:00
@guoshencheng1 #18 如果用的所有开源工具全都去看一遍源码,那每天还干不干活了。。。。
zpf124
    25
zpf124   2018-12-25 10:37:10 +08:00
从逻辑学的角度讲,changelog 里写出来了还能叫做彩蛋么...

不过作为一个给开发者提供的工具,里面加这种东西与其叫菜单不如叫下毒。
SakuraKuma
    26
SakuraKuma   2018-12-25 10:39:24 +08:00
这操作。。
要是对企业,怕是打包走人了。。
某些企业对这些西方节日,十分。。emmm
UIXX
    27
UIXX   2018-12-25 10:40:58 +08:00
很影响 Ant Design 口碑的一次事件,技术的问题可以改,但是信任出现裂缝这是很难修补的。
同时充分暴露出了这个团队的缺点:不够专业、不够负责,没有服务意识等。

一个库开源出来,为一定数量的人造成了影响、也为自己收获了口碑,那这个库作者势必要为自己库代码(尤其是私货)给别人造成麻烦负责的,但“无须体现”让我觉得 Ant Design 掉进了一颗老鼠屎,以后会尽量避免使用这种团队开发的产品。
popu111
    28
popu111   2018-12-25 10:41:34 +08:00
@guoshencheng1 即使看完了,也顶不住半路喂屎啊……你还每天追着 commit 看不成
wengjin456123
    29
wengjin456123   2018-12-25 10:43:32 +08:00
我司也使用 antdesgin,准备慢慢不用了,寻找替代
xiaochocking
    30
xiaochocking   2018-12-25 10:44:01 +08:00
@popu111 需要更新版本的时候再看一遍
leisure
    31
leisure   2018-12-25 10:45:58 +08:00
我们项目还好还没上线....
guoshencheng1
    32
guoshencheng1   2018-12-25 10:46:16 +08:00   ❤️ 2
@popu111 对的,我的意思不是 antd 全无责任,身为开发者的我们对自己的应用足够负责就好了,antd 的源码非常简单易懂,简单看一下不成问题,UI 框架这种东西,用了就很少会更新到最新的,追着 commit 其实没什么必要,如果在使用这种简单易懂的开源框架没有看过源码只是跟风的话,这是对自己的应用的不负责。
jun0205
    33
jun0205   2018-12-25 10:47:39 +08:00 via iPhone
antd 的开发者对使用者都是爱用不用的态度,感觉只是一个内部项目开源出来了,需求都是优先看内部。根本不是一个开源了让社区能参与的项目。
matsuijurina
    34
matsuijurina   2018-12-25 10:48:31 +08:00 via Android   ❤️ 1
如果面对的客户所在的国家和区域宗教信仰不同,这样的彩蛋是完全可能导致公司失去合同,赔偿损失,甚至前端下岗的。有些程序员对于维护这种挂着厂牌的开源项目毫无责任感和敬畏感,始终就是一个玩耍的态度,今天塞个彩蛋代码,明天缺钱了再塞一段挖矿代码,不知道是在砸谁的锅。
DEEPPANDA
    35
DEEPPANDA   2018-12-25 10:49:31 +08:00 via iPhone   ❤️ 6
人家辛辛苦苦开发良心产品开源给大家免费用,对于那些喷子我只想说:爱用就用,不用滚。
hitfm
    36
hitfm   2018-12-25 10:50:28 +08:00
@UIXX 同意,态度很重要。定位为企业级框架,出了事情直接抛一个样式覆盖的方案然后关掉 issue 也太随便了。

刚更新了版本去掉了这彩蛋了: https://github.com/ant-design/ant-design/commit/6550df34b639ab0b3bf2c1cbf9b9828735c1fd41
whileFalse
    37
whileFalse   2018-12-25 10:50:29 +08:00
谁能告我彩蛋的效果是啥
e9e499d78f
    38
e9e499d78f   2018-12-25 10:51:34 +08:00 via iPhone   ❤️ 1
刚觉是射了一坨在 button 上
Vegetable
    40
Vegetable   2018-12-25 10:54:12 +08:00
@DEEPPANDA 这波滚不少,慢慢来
x66
    42
x66   2018-12-25 10:55:02 +08:00
cyspy
    43
cyspy   2018-12-25 10:59:11 +08:00
重要的是这个彩蛋真的很丑,而且大部分人看到这东西的第一反应并不能联想到圣诞节,不然也能少些负面反馈
JoyNeop
    44
JoyNeop   2018-12-25 11:00:15 +08:00
JoyNeop
    45
JoyNeop   2018-12-25 11:00:27 +08:00
搞事搞事 23333
learnshare
    46
learnshare   2018-12-25 11:00:33 +08:00
所以只推荐 Bootstrap
tangler
    47
tangler   2018-12-25 11:01:53 +08:00
tangler
    48
tangler   2018-12-25 11:02:16 +08:00
ayase252
    49
ayase252   2018-12-25 11:04:59 +08:00 via iPhone
之前好像看到过这个彩蛋的讨论,可惜关注度不高。现在果然爆炸了。埋彩蛋可以,但是要有控制手段,要有提前告知。我们大多数时候不是在为自己写程序啊。
Timgle168
    50
Timgle168   2018-12-25 11:05:57 +08:00
坑惨了
mritd
    51
mritd   2018-12-25 11:06:28 +08:00   ❤️ 2
这波操作是真的秀,很早就说: 珍爱生命,远离阿里开源
mritd
    52
mritd   2018-12-25 11:07:19 +08:00
感觉阿里开源面临的不是技术问题,而是信任危机
foolyf
    53
foolyf   2018-12-25 11:08:09 +08:00
阿里开源的东西果然一如既往
FakeLeung
    54
FakeLeung   2018-12-25 11:10:20 +08:00   ❤️ 2
@whileFalse #37


代码里下毒。
JLNR
    55
JLNR   2018-12-25 11:10:39 +08:00   ❤️ 1
阿里的价值观还是一如既往的秀得我头晕
tabris17
    56
tabris17   2018-12-25 11:12:00 +08:00
受害者持续增加……
ChristopherWu
    57
ChristopherWu   2018-12-25 11:12:01 +08:00
@JoyNeop 厉害。。大佬的英语,作为魔法世家的弟子,实在佩服。。
SEARCHINGFREE
    58
SEARCHINGFREE   2018-12-25 11:12:28 +08:00
前端娱乐圈,又添新乐子
PhpBestRubbish
    59
PhpBestRubbish   2018-12-25 11:12:35 +08:00
真的很讨厌这种东西,还有那种举动给用户头像加上圣诞帽的,不是所有人喜欢
wrq
    60
wrq   2018-12-25 11:16:58 +08:00
笑得我肚子疼
zpf124
    61
zpf124   2018-12-25 11:18:17 +08:00   ❤️ 11
@DEEPPANDA 所以只要是辛苦弄得都是对得?

那你们喷之前那几个给 event-stream 加挖矿代码得人干什么?
人家从原作者那接手之后幸幸苦苦维护了一年多,最后加个挖矿代码怎么了?
ballshapesdsd
    62
ballshapesdsd   2018-12-25 11:20:28 +08:00
谢罪警告
zpf124
    63
zpf124   2018-12-25 11:20:34 +08:00
chiou
    64
chiou   2018-12-25 11:21:22 +08:00
秀啊,天秀
dragonszy
    65
dragonszy   2018-12-25 11:22:26 +08:00
咋一看以为是 ant-design-vue,没想到是 ant-design 官方,这真的是大事件了。
plqws
    66
plqws   2018-12-25 11:24:16 +08:00
@guoshencheng1 我吃饭吃到屎,首先得怪我自己没有把饭拿去做色谱分析?
kuxuan
    67
kuxuan   2018-12-25 11:27:24 +08:00
我用得最多的,赶快修复。
ChristopherWu
    68
ChristopherWu   2018-12-25 11:30:45 +08:00   ❤️ 1
http://qianduanyule.club/

前端娱乐圈已更新。 果然永远不会让我失望。。。
OMGZui
    69
OMGZui   2018-12-25 11:31:39 +08:00
不知道怎么评价,请加个春节彩蛋
undeflife
    70
undeflife   2018-12-25 11:33:02 +08:00
Ant Design

An enterprise-class UI design language and React implementation.

这个 enterprise-class UI 是想说 企业级 UI 吗? 有这么个说法?
duan602728596
    71
duan602728596   2018-12-25 11:33:34 +08:00 via iPhone
该用还得用。不想用的,要么自己拿 react-components 封装,要么换 vue 去。
jin5354
    72
jin5354   2018-12-25 11:35:54 +08:00   ❤️ 3
@guoshencheng1 可拉倒吧,这种偷摸塞几行的代码,你不逐行扫源码过去根本发现不了,这跟简不简单没关系,只要代码量大花时间一定不会少,打开你项目的 node_modules 文件夹,告诉我里面代码你审计过多少?你能对你项目负多少责?
littlebaozi
    73
littlebaozi   2018-12-25 11:35:59 +08:00
脑子瓦特了
aino
    74
aino   2018-12-25 11:37:58 +08:00   ❤️ 4
@jianzhihao1996 #3 张谱,前端一枚,老家宁波东钱湖,现在杭州滨江某公司就职。
因为这个彩蛋被裁
zlhsvc
    75
zlhsvc   2018-12-25 11:38:28 +08:00
今天可以给你加个彩蛋,明天就能给你来个后门。
designer
    76
designer   2018-12-25 11:40:51 +08:00 via iPhone   ❤️ 4
“ hello, 大家好,我是一个在伊朗工作的中国前端,因为使用 antd 进行平台开发,在伊朗官方网站上显示出来圣诞快乐,目前已经被抓,求大家救救 me ”
lincanbin
    77
lincanbin   2018-12-25 11:41:00 +08:00
@guoshencheng1 js 的特性,就是自带的库里,什么都没有,一般一个项目 npm install 后就是几百兆的代码。
连个简单的时间格式化都要引用一个库的。
yongzhong
    78
yongzhong   2018-12-25 11:42:30 +08:00
@aino #74 心疼...
OMGZui
    79
OMGZui   2018-12-25 11:42:51 +08:00   ❤️ 1
ericgui
    80
ericgui   2018-12-25 11:45:15 +08:00
这代码有问题啊,就算是判断圣诞,也应该是 12 月 25 日啊
jun0205
    81
jun0205   2018-12-25 11:46:22 +08:00
看了项目正式版里面还是用的老版本,不受这个影响,但是测试项目里面是最近更新的 antd,有受到影响,不过影响小点。
一个有各国不同宗教人使用的项目,要是真受到这个彩蛋影响,问题还是蛮大的。
后续看能不能找项目替代 antd,UI 框架做这种事问题太严重了。
SEARCHINGFREE
    82
SEARCHINGFREE   2018-12-25 11:46:30 +08:00
@ericgui 你一定不是前端
morethansean
    83
morethansean   2018-12-25 11:46:43 +08:00 via Android
@ericgui 一看就不是前端...
nezumi85757
    84
nezumi85757   2018-12-25 11:49:48 +08:00
等等看官方怎么说
dbpe
    85
dbpe   2018-12-25 11:50:06 +08:00
>>Issue 是用来提问题跟解决问题,而不是用来炒热点的,讨论到这份上只能锁了

这个天秀


PS:我感觉是不是内部有矛盾啊....
ChristopherWu
    86
ChristopherWu   2018-12-25 11:50:15 +08:00
@OMGZui 有点乱..
qingzhan
    87
qingzhan   2018-12-25 11:51:42 +08:00
@ericgui 一看就不是前端

我还是觉得应该不是偏右的锅吧~

“早就知道要被骂了” 表达了对傻逼产品的无奈之选~
civet
    88
civet   2018-12-25 11:55:44 +08:00
v2ex.com/t/490917

这个之前就讨论过了啦
Trim21
    89
Trim21   2018-12-25 11:57:47 +08:00
@DEEPPANDA #35 偷偷加东西造成别人损失, 这已经不叫良心了.

而且这么一波下去 antd 估计也没多少人还愿意用了.
ylsc633
    90
ylsc633   2018-12-25 12:00:32 +08:00
@PhpBestRubbish 哈哈 你指的是 掘金吗?? 哈哈哈 最近我发现 这网站给我头像上强行加了个帽子
lincanbin
    91
lincanbin   2018-12-25 12:04:56 +08:00   ❤️ 16


支付宝 2016 年儿童节的彩蛋,是把所有用户的用户名强制改成“**宝宝”。

可见恶心人的彩蛋是阿里一贯的传统,十分符合阿里价值观,应该给埋这个彩蛋的同学升职加薪。
lincanbin
    92
lincanbin   2018-12-25 12:08:06 +08:00   ❤️ 10
thcode
    93
thcode   2018-12-25 12:13:24 +08:00   ❤️ 1
建议 Ant Design 加入愚人节彩蛋,4 月 1 日当天点击按钮随机出现“ you're fooled!”的弹窗
edsheeran
    94
edsheeran   2018-12-25 12:16:21 +08:00 via iPhone   ❤️ 2
@DEEPPANDA 伊朗工作的程序猿用生命寫代碼,而你只是個鍵盤俠
boris1993
    95
boris1993   2018-12-25 12:27:53 +08:00 via Android
@thcode #92 按照这次的尿性,得写成
Hey~ You are FOOLED! ASSHOLE! LOL!
mercury233
    96
mercury233   2018-12-25 12:28:00 +08:00
话说回来,哪天国外开源前端库学 notepad++搞个事情怎么办
DEEPPANDA
    97
DEEPPANDA   2018-12-25 12:29:58 +08:00 via iPhone
@zpf124 请不要偷换概念,修改一下按钮样式和加挖矿木马是两个性质的事情。
edsheeran
    98
edsheeran   2018-12-25 12:33:09 +08:00 via iPhone   ❤️ 7
@DEEPPANDA 不都是偷偷加屎嗎?有什麼不一樣?
TingHaiJamiE
    99
TingHaiJamiE   2018-12-25 12:33:18 +08:00   ❤️ 1
只有圣诞节有,而春节没有,是突显国际化么( MSL 地区 /国家咋办...)。2333333
miyuki
    100
miyuki   2018-12-25 12:38:11 +08:00
antd 牛逼
1  2  3  
关于   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   1301 人在线   最高记录 5168   ·     Select Language
创意工作者们的社区
World is powered by solitude
VERSION: 3.9.8.5 · 159ms · UTC 23:06 · PVG 07:06 · LAX 16:06 · JFK 19:06
♥ Do have faith in what you're doing.