From abf829f542023e089c741c55f73098cae52dcb12 Mon Sep 17 00:00:00 2001
From: wltan <53135010+wltan@users.noreply.github.com>
Date: Tue, 3 Mar 2020 00:00:08 +0800
Subject: [PATCH] Issue #7614: Update doc for OneTopLevelClass

---
 .../checks/design/OneTopLevelClassCheck.java  | 22 +++++-----
 src/xdocs/config_design.xml                   | 40 +++++++++++--------
 2 files changed, 34 insertions(+), 28 deletions(-)

diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/checks/design/OneTopLevelClassCheck.java b/src/main/java/com/puppycrawl/tools/checkstyle/checks/design/OneTopLevelClassCheck.java
index a2493391186..439205dbda7 100644
--- a/src/main/java/com/puppycrawl/tools/checkstyle/checks/design/OneTopLevelClassCheck.java
+++ b/src/main/java/com/puppycrawl/tools/checkstyle/checks/design/OneTopLevelClassCheck.java
@@ -39,7 +39,7 @@
  * public class, enum or interface, top-level type is the first type in file.
  * </p>
  * <p>
- * An example of check's configuration:
+ * To configure the check:
  * </p>
  * <pre>
  * &lt;module name=&quot;OneTopLevelClass&quot;/&gt;
@@ -52,32 +52,32 @@
  * An example of code with violations:
  * </p>
  * <pre>
- * public class Foo{
- *   //methods
+ * public class Foo { // OK, first top-level class
+ *   // methods
  * }
  *
- * class Foo2{
- *   //methods
+ * class Foo2 { // violation, second top-level class
+ *   // methods
  * }
  * </pre>
  * <p>
  * An example of code without public top-level type:
  * </p>
  * <pre>
- * class Foo{ // top-level class
- *   //methods
+ * class Foo { // OK, first top-level class
+ *   // methods
  * }
  *
- * class Foo2{
- *   //methods
+ * class Foo2 { // violation, second top-level class
+ *   // methods
  * }
  * </pre>
  * <p>
  * An example of code without violations:
  * </p>
  * <pre>
- * public class Foo{
- *   //methods
+ * public class Foo { // OK, only one top-level class
+ *   // methods
  * }
  * </pre>
  *
diff --git a/src/xdocs/config_design.xml b/src/xdocs/config_design.xml
index aac86ae9cd6..38226ea2917 100644
--- a/src/xdocs/config_design.xml
+++ b/src/xdocs/config_design.xml
@@ -666,7 +666,7 @@ public class StringUtils // not final to allow subclassing
 
       <subsection name="Examples" id="OneTopLevelClass_Examples">
         <p>
-          An example of check's configuration:
+          To configure the check:
         </p>
         <source>
 &lt;module name="OneTopLevelClass"/&gt;
@@ -678,35 +678,41 @@ public class StringUtils // not final to allow subclassing
         <p>
           An example of code with violations:
         </p>
-        <source>
-public class Foo{
-  //methods
+        <div class="wrapper">
+          <pre>
+public class Foo { // OK, first top-level class
+  // methods
 }
 
-class Foo2{
-  //methods
+class Foo2 { // violation, second top-level class
+  // methods
 }
-        </source>
+          </pre>
+        </div>
         <p>
           An example of code without public top-level type:
         </p>
-        <source>
-class Foo{ // top-level class
-  //methods
+        <div class="wrapper">
+          <pre>
+class Foo { // OK, first top-level class
+  // methods
 }
 
-class Foo2{
-  //methods
+class Foo2 { // violation, second top-level class
+  // methods
 }
-        </source>
+          </pre>
+        </div>
         <p>
           An example of code without violations:
         </p>
-        <source>
-public class Foo{
-  //methods
+        <div class="wrapper">
+          <pre>
+public class Foo { // OK, only one top-level class
+  // methods
 }
-        </source>
+          </pre>
+        </div>
       </subsection>
 
       <subsection name="Example of Usage" id="OneTopLevelClass_Example_of_Usage">