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
update jquery, jquery-migrate, and jquery-ui #2482
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good, thanks!
plugins/javadoc/src/main/kotlin/org/jetbrains/dokka/javadoc/renderer/KorteJavadocRenderer.kt
Show resolved
Hide resolved
@@ -5,8 +5,8 @@ | |||
<link rel="stylesheet" type="text/css" href="{{ pathToRoot }}stylesheet.css" title="Style"> | |||
<link rel="stylesheet" type="text/css" href="{{ pathToRoot }}dokka-javadoc-stylesheet.css" title="Style"> | |||
<link rel="stylesheet" type="text/css" href="{{ pathToRoot }}jquery/jquery-ui.css" title="Style"> | |||
<script type="text/javascript" src="{{ pathToRoot }}jquery/jquery-3.3.1.js"></script> | |||
<script type="text/javascript" src="{{ pathToRoot }}jquery/jquery-migrate-3.0.1.js"></script> | |||
<script type="text/javascript" src="{{ pathToRoot }}jquery/jquery-3.5.0.js"></script> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you look inside jquery-3.5.0.js
, you'll find that it's jquery 3.6.0
. Can you please rename the file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suppose {{ jQueryVersion() }}
should be used here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It makes sense to add the values back in KorteJavadocRenderer.kt
and use them here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately, Dokka had a development version of jQuery instead of a minified production one.
@@ -5,8 +5,8 @@ | |||
<link rel="stylesheet" type="text/css" href="{{ pathToRoot }}stylesheet.css" title="Style"> | |||
<link rel="stylesheet" type="text/css" href="{{ pathToRoot }}dokka-javadoc-stylesheet.css" title="Style"> | |||
<link rel="stylesheet" type="text/css" href="{{ pathToRoot }}jquery/jquery-ui.css" title="Style"> | |||
<script type="text/javascript" src="{{ pathToRoot }}jquery/jquery-3.3.1.js"></script> | |||
<script type="text/javascript" src="{{ pathToRoot }}jquery/jquery-migrate-3.0.1.js"></script> | |||
<script type="text/javascript" src="{{ pathToRoot }}jquery/jquery-3.5.0.js"></script> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suppose {{ jQueryVersion() }}
should be used here.
I can also make it a non-development version. |
update to jquery 3.5.0 update to jquery-migrate 3.4.0 update to jquery-ui 1.13.1 closes Kotlin#1938
update to jquery 3.6.0 re-add versions in KorteJavadocRenderer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thank you for the contribution!
update to jquery 3.5.0
update to jquery-migrate 3.4.0
update to jquery-ui 1.13.1
closes #1938