Skip to content

refactor hook and support ref #47

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

Merged
merged 10 commits into from
May 17, 2020
Merged

refactor hook and support ref #47

merged 10 commits into from
May 17, 2020

Conversation

hengkx
Copy link
Member

@hengkx hengkx commented May 17, 2020

准备移除废弃api ant-design/ant-design#22493

@hengkx
Copy link
Member Author

hengkx commented May 17, 2020

@afc163 这个ci是配置有问题把

@afc163
Copy link
Member

afc163 commented May 17, 2020

配置没问题,仔细观察一下。

image

@hengkx
Copy link
Member Author

hengkx commented May 17, 2020

都没有hsx的文件吧

@hengkx
Copy link
Member Author

hengkx commented May 17, 2020

jsx

@afc163
Copy link
Member

afc163 commented May 17, 2020

需要加上 tsx

@afc163
Copy link
Member

afc163 commented May 17, 2020

改成这样吧,再把出现的问题顺便修掉。

-    "lint": "eslint src/ --ext .jsx,.js,.md",
+    "lint": "eslint .",

@codecov
Copy link

codecov bot commented May 17, 2020

Codecov Report

Merging #47 into master will increase coverage by 10.71%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##           master       #47       +/-   ##
============================================
+ Coverage   89.28%   100.00%   +10.71%     
============================================
  Files           1         1               
  Lines          56        43       -13     
  Branches       13        12        -1     
============================================
- Hits           50        43        -7     
+ Misses          6         0        -6     
Impacted Files Coverage Δ
src/index.tsx 100.00% <100.00%> (+10.71%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b86c07a...a0472cb. Read the comment docs.

@afc163
Copy link
Member

afc163 commented May 17, 2020

覆盖率也顺便提上来吧,应该就两三行没覆盖到。

@hengkx hengkx requested a review from afc163 May 17, 2020 07:31
@hengkx
Copy link
Member Author

hengkx commented May 17, 2020

刚刚有其它依赖 更新了吗??
image

@hengkx hengkx requested a review from afc163 May 17, 2020 08:05
@afc163
Copy link
Member

afc163 commented May 17, 2020

eslint ignore 掉就行

@afc163 afc163 merged commit 4d89a43 into react-component:master May 17, 2020
@hengkx
Copy link
Member Author

hengkx commented May 17, 2020

发个版本

@afc163
Copy link
Member

afc163 commented May 17, 2020

@hengkx
Copy link
Member Author

hengkx commented May 17, 2020

2.1.0就可以了把 没有做 不兼容的变更

@afc163
Copy link
Member

afc163 commented May 17, 2020

2.x 理论上 React 15 还能用。react-component/progress#82

if (autoFocus && !disabled) {
this.focus();
focus();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个 focus 是无源之水……

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

我处理下 ,批量替换了this,... 竟然不报错

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

单元测试 覆盖到了,为什么会通过

}
}
}, [props.autoFocus, props.disabled]);
Copy link
Member

@zombieJ zombieJ May 18, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

disabled 切换会反复 autoFocus.

@hengkx hengkx mentioned this pull request May 18, 2020
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

Successfully merging this pull request may close these issues.

3 participants