-
Notifications
You must be signed in to change notification settings - Fork 843
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
Export missing type definitions #2434
Conversation
In Cloud UI linter is also complaining about |
Tested with
line 3129 is
from
I'm looking into this further. |
This rings a bell. The d.ts generator attempts to reuse types if they are identical (or something like that) |
You appear to be correct. I don't know what layer (TypeScript or dts-generator) does that, but yep. What's happening is it tries to re-use ButtonIconSide from |
…le instead of index.ts
Pushed a fix for the
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Built eui.d.ts
from this PR and pulled it into an empty TypeScript project. I confirmed the ability to import the added/corrected types.
import {
ButtonSize,
ButtonIconSide,
EuiButtonEmptyColor,
EuiButtonEmptySizes,
EuiButtonIconColor,
EuiLinkColor,
EuiLinkType,
} from '@elastic/eui';
Summary
Exports formerly exported types:
ButtonSize
ButtonIconSide
EmptyButtonColor
(now[nowEuiEmptyButtonColor
)EuiButtonEmptyColor
]EmptyButtonSizes
(now[nowEuiEmptyButtonSizes
)EuiButtonEmptySizes
]ButtonIconColor
(nowEuiButtonIconColor
)EuiLinkColor
EuiLinkType
Checklist
- [ ] Checked in dark mode- [ ] Checked in mobile- [ ] Checked in IE11 and Firefox- [ ] Props have proper autodocs- [ ] Added documentation examples- [ ] Added or updated jest tests- [ ] Checked for breaking changes and labeled appropriately- [ ] Checked for accessibility including keyboard-only and screenreader modes