search

umijs / plugins

🍣 The one-stop shop for official Umi plugins and presets.
348 stars 238 forks source link

plugin-layout 的两个问题 #100

Open sorrycc opened 4 years ago

sorrycc commented 4 years ago

两个问题，

  1. 渲染两次（初始化时，以及 HMR 时）
  2. componentWillMount renamed 警告信息

image

复现步骤，

  1. 找个空目录
  2. 安装依赖，yarn add umi @umijs/playout-layout
  3. 生成最简页面，npx umi g page index
  4. 配置，echo 'export default {layout:{}}' > .umirc.ts
  5. 编辑 pages/index.js，函数体内加入一行 console.log(1);
  6. npx umi dev
Ariel-Cheng commented 4 years ago
  1. pro-layout 也有要一起看下
  2. componentWillMount renamed 警告信息 : https://github.com/ant-design/ant-design-pro-layout/issues/127
chenshuai2144 commented 4 years ago

警告信息来自 https://github.com/nfl/react-helmet/issues/426，下个版本根据 issue中的指引修改为 react-helmet-async

问题来自这里 https://github.com/ant-design/ant-design-pro-layout/blob/3d244e0f0e8de3528e4a4549cccd5452eae942b9/src/WrapContent.tsx#L79

是为了解决 layout 高度计算失败引入的。