Skip to content
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

Invalid default platform tab content on page load for merged declarations #2801

Open
atyrin opened this issue Jan 9, 2023 · 0 comments
Open
Labels

Comments

@atyrin
Copy link
Contributor

atyrin commented Jan 9, 2023

My minimal reproducer includes:

  • multiplatform library
  • expect/actual declaration
  • extra overload for one platform

Sample:

// common
expect class C() {
    /**
     * Fun f for common
     */
    fun f(p1: String)
}

// JVM
actual class C actual constructor() {
    /**
     * Fun f for JVM
     */
    actual fun f(p1: String){}
    fun f(p1: Int){}
}

// another platform
actual class C actual constructor() {
    /**
     * Fun f for another
     */
    actual fun f(p1: String){}
}

On the first opening of the class C page we will see (content for JVM is shown under Common)
image

After the tabs switching the content changed to expected:
image

N.B: it depends on how the function overridden, e.g. if we add fun f(p1: String, p2: Short){} everything will be fine

Dokka: 1.7.20
Option mergeImplicitExpectActualDeclarations = true is used

@atyrin atyrin added the bug label Jan 9, 2023
@atyrin atyrin changed the title Invalid platform tab content on page load Invalid default platform tab content on page load for merged declarations Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant