8268557: Module page uses unstyled table class

hns · hns · commit 38260122815a · 2021-06-30T18:40:17.000Z
Reviewed-by: jjg
diff --git a/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/resources/stylesheet.css b/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/resources/stylesheet.css
@@ -350,14 +350,12 @@ ul.see-list-long li:not(:last-child):after {
 /*
  * Styles for tables.
  */
-.summary-table {
+.summary-table, .details-table {
     width:100%;
     border-spacing:0;
     border-left:1px solid #EEE;
     border-right:1px solid #EEE;
     border-bottom:1px solid #EEE;
-}
-.summary-table {
     padding:0;
 }
 .caption {
@@ -445,7 +443,7 @@ div.table-tabs > button.table-tab {
         grid-template-columns: minmax(15%, max-content) minmax(15%, auto);
     }
 }
-.summary-table > div {
+.summary-table > div, .details-table > div {
     text-align:left;
     padding: 8px 3px 3px 7px;
 }
diff --git a/test/langtools/jdk/javadoc/doclet/checkStylesheetClasses/CheckStylesheetClasses.java b/test/langtools/jdk/javadoc/doclet/checkStylesheetClasses/CheckStylesheetClasses.java
@@ -120,7 +120,7 @@ void run() throws Exception {
                 "modifiers", "permits", "return-type");
 
         // misc: these are defined in HtmlStyle, and used by the doclet
-        removeAll(htmlStyleNames, "col-plain", "details-table", "external-link",
+        removeAll(htmlStyleNames, "col-plain", "external-link",
                 "hierarchy", "index", "package-uses", "packages", "permits-note",
                 "serialized-package-container", "source-container");
 
diff --git a/test/langtools/jdk/javadoc/doclet/testStylesheet/TestStylesheet.java b/test/langtools/jdk/javadoc/doclet/testStylesheet/TestStylesheet.java
@@ -140,7 +140,7 @@ public void test(Path base) {
                         overflow: auto;
                     }""",
                 """
-                    .summary-table > div {
+                    .summary-table > div, .details-table > div {
                         text-align:left;
                         padding: 8px 3px 3px 7px;
                     }""",

Original file line number	Diff line number	Diff line change
`@@ -350,14 +350,12 @@ ul.see-list-long li:not(:last-child):after {`
`350`	`350`	`/*`
`351`	`351`	`* Styles for tables.`
`352`	`352`	`*/`
`353`		`-.summary-table {`
	`353`	`+.summary-table, .details-table {`
`354`	`354`	`width:100%;`
`355`	`355`	`border-spacing:0;`
`356`	`356`	`border-left:1px solid #EEE;`
`357`	`357`	`border-right:1px solid #EEE;`
`358`	`358`	`border-bottom:1px solid #EEE;`
`359`		`-}`
`360`		`-.summary-table {`
`361`	`359`	`padding:0;`
`362`	`360`	`}`
`363`	`361`	`.caption {`
`@@ -445,7 +443,7 @@ div.table-tabs > button.table-tab {`
`445`	`443`	`grid-template-columns: minmax(15%, max-content) minmax(15%, auto);`
`446`	`444`	`}`
`447`	`445`	`}`
`448`		`-.summary-table > div {`
	`446`	`+.summary-table > div, .details-table > div {`
`449`	`447`	`text-align:left;`
`450`	`448`	`padding: 8px 3px 3px 7px;`
`451`	`449`	`}`