Move inheritors, params, see also and samples tabs to description for classlikes

[atyrin]

Migrate params, seealso, samples, and inheritors to the description

• remove contentForComments API, those tags will be handled in the contentForDescription
• change API for contentForScopes: remove inheritors parameter in the method
• remove inheritors from the package list: I didn't find a way to show them there. mat be something with markdown but I don't see a reason to keep that.

Fix #2688

[atyrin]

There is still an issue with the case:

mergeImplicitExpectActualDeclarations=true

// jvm
open class A
class J1 : A()

// linuxX64
open class A
class L1 : A()

But seems it has to be addressed on the site where the inheritors list is combined.

[atyrin]

• example of the problem with merged inheritors: https://dokka-snapshots.s3.eu-central-1.amazonaws.com/tabs-to-description/stdlib/932a295/kotlin-stdlib/kotlin/-comparable/index.html
  You may see the "real" inheritors with link highlighting

• Example of "properly merged" platforms that : https://dokka-snapshots.s3.eu-central-1.amazonaws.com/tabs-to-description/stdlib/932a295/kotlin-stdlib/kotlin/-char-sequence/index.html. But even this is based on the dirty hack

• Example of inheritors representation "as expected": https://dokka-snapshots.s3.eu-central-1.amazonaws.com/tabs-to-description/stdlib/932a295/kotlin-stdlib/kotlin.collections/-abstract-mutable-map/index.html. Here only JS implementation has HashMap inheritor.

UPD: Fixed

[vmishenev]

sourceSets = availableSourceSets?

[atyrin]

In that case, for MPP all samples will be shown on each platform tab

[atyrin]

At least the behavior is preserved as it was before. So no performance degradation is expected in this area.

[IgnatBeresnev]

Well done — especially on the number of tests! :) And thanks for the PR and in-code comments, they made it easier to understand changes, corner cases and made review easier overall 👍

The only comment I didn't understand was the following:

remove inheritors from the package list

Which package list do you mean? Could you share an example of before/after, if it's still relevant?

[atyrin]

@IgnatBeresnev

The only comment I didn't understand was the following:

remove inheritors from the package list

Which package list do you mean? Could you share an example of before/after, if it's still relevant?

I've meant this change: https://github.com/Kotlin/dokka/pull/2749/files#diff-393cd6c01196a89045f20b4a981cf4c5b5ac45cec583c8bc2fb41ea0e9fa1765R267
This precise contentForScope is used in contentForPackage. I removed all attempts here to show inheritors. For classlikes it moved to description. And for packages just removed.

[IgnatBeresnev]

👍 👍