Skip to content
This repository has been archived by the owner on Mar 7, 2024. It is now read-only.

FIX WebStrom 组件无法自动提示属性 #1203

Merged
merged 7 commits into from
Aug 5, 2020
Merged

Conversation

coffee377
Copy link
Contributor

@coffee377 coffee377 commented Aug 2, 2020

显示声明组件类型,以修复 tsc 编译时自动类型推导生成的类型文件导致 WebStorm 组件不能自动提示

fix-tip

@vercel
Copy link

vercel bot commented Aug 2, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/remaxjs/remax/h7wv1liv1
✅ Preview: https://remax-git-fork-coffee377-dts.remaxjs.vercel.app

@codesandbox-ci
Copy link

codesandbox-ci bot commented Aug 2, 2020

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit b57659d:

Sandbox Source
remaxjs/examples: one Configuration

@codecov
Copy link

codecov bot commented Aug 2, 2020

Codecov Report

Merging #1203 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1203      +/-   ##
==========================================
+ Coverage   95.26%   95.28%   +0.01%     
==========================================
  Files         350      350              
  Lines        3698     3734      +36     
  Branches      547      563      +16     
==========================================
+ Hits         3523     3558      +35     
- Misses        173      174       +1     
  Partials        2        2              
Impacted Files Coverage Δ
...ax-one/src/hostComponents/Navigator/props/index.ts 100.00% <ø> (ø)
...s/remax-one/src/hostComponents/Text/props/index.ts 100.00% <ø> (ø)
...kages/remax-ali/src/hostComponents/Button/index.ts 100.00% <100.00%> (ø)
...kages/remax-ali/src/hostComponents/Canvas/index.ts 100.00% <100.00%> (ø)
...ges/remax-ali/src/hostComponents/Checkbox/index.ts 100.00% <100.00%> (ø)
...emax-ali/src/hostComponents/CheckboxGroup/index.ts 100.00% <100.00%> (ø)
...emax-ali/src/hostComponents/ContactButton/index.ts 100.00% <100.00%> (ø)
...s/remax-ali/src/hostComponents/CoverImage/index.ts 100.00% <100.00%> (ø)
...es/remax-ali/src/hostComponents/CoverView/index.ts 100.00% <100.00%> (ø)
...ackages/remax-ali/src/hostComponents/Form/index.ts 100.00% <100.00%> (ø)
... and 106 more

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 ed70dff...b57659d. Read the comment docs.

@Darmody
Copy link
Contributor

Darmody commented Aug 3, 2020

这像是 webStorm 的问题?

@coffee377
Copy link
Contributor Author

coffee377 commented Aug 3, 2020

这像是 webStorm 的问题?

我用 WS 开发的,所以标题就这样写,在 VS 中 也有同样的提示问题,如果类型声明文件不正确的话,是不能智能提示的

@@ -9,6 +9,7 @@ module.exports = {
'pages/turbo-page/index',
Copy link
Member

Choose a reason for hiding this comment

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

e2e 下面的改动就不要提交了。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

好的,我来去掉

@yesmeck
Copy link
Member

yesmeck commented Aug 4, 2020

@coffee377
Copy link
Contributor Author

以后是不是不需要这样配置了 https://remaxjs.org/faq#webstorm-ide-%E7%BC%96%E8%BE%91%E5%99%A8%E4%B8%8B-api-%E7%B1%BB%E5%9E%8B%E6%97%A0%E6%8F%90%E7%A4%BA%E9%97%AE%E9%A2%98

组件API 直接就会有相应的提示,在 WebStrom 2019.3 ,2020.2 均测试过

@yesmeck
Copy link
Member

yesmeck commented Aug 5, 2020

@coffee377 那顺便把 FAQ 里的这个问题也删了吧

This reverts commit c856612
@coffee377
Copy link
Contributor Author

@coffee377 那顺便把 FAQ 里的这个问题也删了吧

@yesmeck 我好像没办法删吧!

@coffee377
Copy link
Contributor Author

@yesmeck 明白你的意思了,是让我把 doc 中 FAQ 相关的给删掉吧

@yesmeck
Copy link
Member

yesmeck commented Aug 5, 2020 via email

@wyx1818
Copy link

wyx1818 commented Aug 18, 2020

大佬,WS里开发的话,需要装哪些东西呢

@coffee377
Copy link
Contributor Author

大佬,WS里开发的话,需要装哪些东西呢

@wyx1818 都是常规依赖,你需要什么就添加响应依赖就可以了

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

Successfully merging this pull request may close these issues.

None yet

4 participants