issue: ENG-9039
Details
Details
Please see the video (this is with longer duration):
Here is duration 150:
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
| native/tooltip/nux-tips-overlay.react.js | ||
|---|---|---|
| 9–12 | Those types are used in worklets. We have to import them for flow, but eslint doesn't understand that | |
| 84 | I don't think this value can ba calculated in any way? I think I just have to use something that looks rigth? | |
| 202–207 | For community drawer button which is rather small, the hardcoded 20 made it look awkward. I used 20% instead, but this may be changed later | |
| 228–230 | In worklets we cannot use flow types | |
| native/tooltip/nux-tips-overlay.react.js | ||
|---|---|---|
| 9–12 | It seems like it can't be fixed on eslint level https://github.com/jsx-eslint/eslint-plugin-react/issues/1714. It needs to be fixed on babel level - I found an issue https://github.com/babel/babel/issues/15061 where someone is using flowComments plugin. Maybe we should give it a try. https://linear.app/comm/issue/ENG-9040/check-if-flowcomments-babel-plugin-allows-flow-comment-syntax | |
| 84 | Isn't it related to the height of the tooltip? | |
| 202–207 | Please simplify these expressions | |