xiaody
xiaody
Ah ">= 16" is intended for developing this package. I don't think the users need this. Did you enable the npm `engine-strict` config?
Just leave it as default (false). I've noticed that you are using yarn instead of npm. yarn is by default stricter about `engines`. It gives you errors unlike npm's warns....
@dtinth @tuantv413 You are right. `[email protected]` is released without `engines` field.
I'm not familiar with typed React. PR would be very welcomed.
@gareys good catch about the web font issue. But I don't think this component should provide a timeout param, which would be quite weird. You can set a timeout outside...
That sounds weird. Can you provide a online reproduction? https://stackblitz.com/edit/react-lines-ellipsis-2fjoj4 you can fork and edit this.
What about make the line **bold**? PR welcomed.
Nothing looks wrong. The `LinesEllipsis-canvas` elements are used to calculate text layout and are expected to be invisible. What's bothering you?
Thanks @JoaoRPF I have just commented the issue. I think it's unnecessary for this component to clamp the width. If you still have trouble with the latest version, feel free...
React's `invokeGuardedCallbackImpl` [uses such pattern](https://github.com/facebook/react/blob/4e6eec69be632c0c0177c5b1c8a70397d92ee181/packages/shared/invokeGuardedCallbackImpl.js#L190-L195), it creates a `fakeNode`, add a listener on it and dispatch an event synchronously. #41506(with old version antd) maybe a real world case as I...