Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem work with Preact #1839

Closed
paranoidjk opened this issue Sep 12, 2017 · 5 comments
Closed

Problem work with Preact #1839

paranoidjk opened this issue Sep 12, 2017 · 5 comments

Comments

@warmhug
Copy link
Contributor

warmhug commented Sep 18, 2017

preact 在 componentWillReceiveProps 里的问题

componentWillReceiveProps(nextProps) {
  const curProps = this.props;
  this.setState((state, props) => {
    console.log(props === nextProps); // in react.js, it's true.
    console.log(props === curProps); // in preact.js, it's true.
  });
}

ref: react-component/m-list-view@73c317b

warmhug added a commit to react-component/m-pull-to-refresh that referenced this issue Oct 11, 2017
@silentcloud silentcloud added Usage and removed bug labels Oct 13, 2017
@silentcloud silentcloud reopened this Oct 13, 2017
@silentcloud silentcloud removed the Usage label Oct 13, 2017
@silentcloud
Copy link
Contributor

silentcloud commented Oct 18, 2017

  • popover 在 preact android 下面 click 问题 (trigger 的 document click 执行两次)

warmhug added a commit to react-component/m-trigger that referenced this issue Oct 18, 2017
warmhug added a commit to react-component/m-pull-to-refresh that referenced this issue Oct 19, 2017
@ant-design ant-design deleted a comment from ant-design-bot Oct 19, 2017
@warmhug
Copy link
Contributor

warmhug commented Dec 26, 2017

  • preact 把 className 转换为 class 添加到 dom ,导致 重复添加 css name 错误,见 InputItem 组件

@warmhug warmhug reopened this Dec 26, 2017
@ant-design ant-design deleted a comment from paranoidjk Dec 26, 2017
@paranoidjk
Copy link
Contributor Author

@zhang740preactjs/preact-compat#450 关联的 issue 添加过来

@paranoidjk
Copy link
Contributor Author

preactjs/preact#974

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants