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

feat: reworked footer according to new UI #3948

Merged
merged 2 commits into from
Jan 21, 2025
Merged

Conversation

berezinant
Copy link
Contributor

@berezinant berezinant commented Dec 6, 2024

This is how it looks like
image
and this is an example with a custom content
image

@berezinant berezinant force-pushed the ktl-1696-footer branch 3 times, most recently from 57567bc to c37f669 Compare December 9, 2024 15:01
@berezinant berezinant force-pushed the ktl-1696-footer branch 2 times, most recently from 5cc855c to 0b58a9c Compare January 20, 2025 13:31
@berezinant berezinant marked this pull request as ready for review January 21, 2025 14:34
@berezinant berezinant requested a review from krutilov January 21, 2025 15:13
@@ -13,7 +13,7 @@ tasks.withType<AbstractDokkaTask>().configureEach {
@Language("JSON")
val dokkaBaseConfiguration = """
{
"footerMessage": "(c) 2024 My custom footer message",
"footerMessage": "<div class='kt-footer__license'>Kotlin™ is protected under the <a href='https://kotlinlang.org/foundation/kotlin-foundation.html' data-test='external-link ' target='_blank' rel='noreferrer noopener' class='kt-footer__link'>Kotlin&nbsp;Foundation</a> and licensed under the&nbsp;<a href='https://github.com/JetBrains/kotlin-web-site/blob/master/LICENSE' data-test='external-link ' target='_blank' rel='noreferrer noopener' class='kt-footer__link'>Apache&nbsp;2 license</a>. </div>",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure about having full markup here. Is there any other place to put this template?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or it's a test snapsthot?

Copy link
Contributor Author

@berezinant berezinant Jan 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or it's a test snapsthot?

Yes, this is a more complex example of footer customisation for our testing purposes

@berezinant berezinant merged commit f4c4a41 into master Jan 21, 2025
14 checks passed
@berezinant berezinant deleted the ktl-1696-footer branch January 21, 2025 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants