react-component
fix: opt out of ssr full to avoid initial flashing
背景: horizontal responsive模式下, 当全部 menu item 总宽度超出 overflow 宽度时, 默认全部 menu item 都会被绘制到页面, 出现溢出, 随后才会计算和隐藏.
期许: rc-menu 可能仅在 client 端使用, 这个时候, 希望ssr 值非写死, 实现 menu item 当被绘制时是已经计算好的数量, 不出现目前的闪动.
复现方式: 可以用 antd menu 组件的第一个用例复现. 效果如下:
https://github.com/react-component/menu/assets/26617824/061f8e89-c2de-4366-8bb8-ad3b55322fa6
代码改动简述: 在不变动 rc-menu 默认 ssr 设置 “full“ 的前提下, 加入新prop来支持跳出该默认设置.
Codecov Report
Merging #665 (5049d03) into master (89c09ad) will increase coverage by
0.00%. The diff coverage is100.00%.
:exclamation: Current head 5049d03 differs from pull request most recent head 9143b72. Consider uploading reports for the commit 9143b72 to get more accurate results
@@ Coverage Diff @@
## master #665 +/- ##
=======================================
Coverage 99.57% 99.57%
=======================================
Files 27 27
Lines 713 714 +1
Branches 195 193 -2
=======================================
+ Hits 710 711 +1
Misses 3 3
| Files | Coverage Δ | |
|---|---|---|
| src/Menu.tsx | 98.66% <100.00%> (+<0.01%) |
:arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
![codecov[bot] avatar](https://avatars.githubusercontent.com/in/254?v=4 "Published by a pub.dev verified publisher"){kind=small}
直接裁剪的话，会导致 SEO 里没有东西。所以我特地加上了 SSR full 的。如果是这样，干脆 overflow 里加个逻辑，ssr 的时候渲染元素但是不占宽度，client 渲染在放出来？
好哦, 这样的话 overflow 那里 display 计算逻辑可能要小改一下, 因为首次收集到的 menu items 的宽度数据会有误. 我关掉这条提个新的去 overflow?
好哦, 这样的话 overflow 那里 display 计算逻辑可能要小改一下, 因为首次收集到的 menu items 的宽度数据会有误. 我关掉这条提个新的去 overflow?
可以直接改, 只要没有混合其他内容就行, 新开也可.
@yoyo837 如果这个pr ok可以先review这条不？另外那个后面再看
如果那个可以代替这个, 那我建议直接CR那个, 目前假期中, 请稍等哈.