JetBrains / kotlin-wrappers

Kotlin wrappers for popular JavaScript libraries
Apache License 2.0
1.33k stars 165 forks source link

Issue with importing/ using ClickAwayListener #2177

Closed lsafer-meemer closed 5 months ago

lsafer-meemer commented 7 months ago

Is your feature request related to a problem? Please describe. Importing ClickAwayListener returns undefined at runtime due to mui/base does not export it as a default import. Instead, as a named export.

Plus: Someone should check if other mui/base components suffers the same issue

Describe the solution you'd like Change @JsName("default") to @JsName("ClickAwayListener")

Describe alternatives you've considered My correct temporary fix:

@JsNonModule
@JsModule("@mui/base/ClickAwayListener")
private external object ClickAwayListenerModule {
    @JsName("ClickAwayListener")
    val ClickAwayListener: FC<ClickAwayListenerProps>
}

Additional context Issue: [base-ui][ClickAwayListener] Trouble with importing it

https://github.com/JetBrains/kotlin-wrappers/blob/8b491da103de3094690d193fdd69f25523d7e025/kotlin-mui-base/src/jsMain/generated/mui/base/ClickAwayListener.kt#L38C1-L51C65

lucgirardin commented 7 months ago

This may very be related to my issue, which persists since pre.618: https://github.com/JetBrains/kotlin-wrappers/issues/2123

lucgirardin commented 5 months ago

I believe this issue has been resolved by pre-690.