Skip to content

Commit 0ec1d63

Browse files
committedOct 12, 2020
8253117: Replace HTML tables in javadoc summaries with CSS grid elements
Reviewed-by: jjg
1 parent 54bbe76 commit 0ec1d63

File tree

61 files changed

+1828
-2224
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

61 files changed

+1828
-2224
lines changed
 

‎src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/AbstractMemberWriter.java

+1-2
Original file line numberDiff line numberDiff line change
@@ -313,9 +313,8 @@ protected void addUseInfo(List<? extends Element> members, Content heading, Cont
313313
return;
314314
}
315315
boolean printedUseTableHeader = false;
316-
Table useTable = new Table(HtmlStyle.useSummary, HtmlStyle.summaryTable)
316+
Table useTable = new Table(HtmlStyle.summaryTable)
317317
.setCaption(heading)
318-
.setRowScopeColumn(1)
319318
.setColumnStyles(HtmlStyle.colFirst, HtmlStyle.colSecond, HtmlStyle.colLast);
320319
for (Element element : members) {
321320
TypeElement te = (typeElement == null)

‎src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/AllClassesIndexWriter.java

+2-4
Original file line numberDiff line numberDiff line change
@@ -120,9 +120,8 @@ protected void buildAllClassesFile() throws DocFileIOException {
120120
* @param content HtmlTree content to which the links will be added
121121
*/
122122
protected void addContents(Content content) {
123-
Table table = new Table(HtmlStyle.typeSummary, HtmlStyle.summaryTable)
123+
Table table = new Table(HtmlStyle.summaryTable)
124124
.setHeader(new TableHeader(contents.classLabel, contents.descriptionLabel))
125-
.setRowScopeColumn(1)
126125
.setColumnStyles(HtmlStyle.colFirst, HtmlStyle.colLast)
127126
.setId("all-classes-table")
128127
.setDefaultTab(resources.getText("doclet.All_Classes"))
@@ -131,8 +130,7 @@ protected void addContents(Content content) {
131130
.addTab(resources.enumSummary, utils::isEnum)
132131
.addTab(resources.exceptionSummary, e -> utils.isException((TypeElement)e))
133132
.addTab(resources.errorSummary, e -> utils.isError((TypeElement)e))
134-
.addTab(resources.annotationTypeSummary, utils::isAnnotationType)
135-
.setTabScript(i -> "show(" + i + ");");
133+
.addTab(resources.annotationTypeSummary, utils::isAnnotationType);
136134
for (Character unicode : indexBuilder.getFirstCharacters()) {
137135
for (IndexItem indexItem : indexBuilder.getItems(unicode)) {
138136
TypeElement typeElement = (TypeElement) indexItem.getElement();

0 commit comments

Comments
 (0)