zentol commented on a change in pull request #14903:
URL: https://github.com/apache/flink/pull/14903#discussion_r572422797
##########
File path:
flink-docs/src/main/java/org/apache/flink/docs/rest/RestAPIDocGenerator.java
##########
@@ -220,32 +220,42 @@ private static String createHtmlEntry(MessageHeaders<?,
?, ?> spec) {
{
sb.append(" <tr>\n");
sb.append(" <td colspan=\"2\">\n");
- sb.append(
- " <button data-toggle=\"collapse\" data-target=\"#"
- + reqHash
- + "\">Request</button>\n");
- sb.append(" <div id=\"" + reqHash + "\"
class=\"collapse\">\n");
+ sb.append(" <div class=\"book-expand\">\n");
+ sb.append(" <label>\n");
+ sb.append(" <div class=\"book-expand-head flex
justify-between\">\n");
+ sb.append(" <span>Request</span>\n");
+ sb.append(" <span>...</span>\n");
Review comment:
actually, why not just a single `span` without the `justify-between`?
Having the dots/arrow not jump to the right when opened seems nicer?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
